kepler icon indicating copy to clipboard operation
kepler copied to clipboard

CI post action enhance

Open SamYuan1990 opened this issue 3 years ago • 3 comments

Is your feature request related to a problem? Please describe.

  • remove post integration test
  • image build from base to kepler

Describe the solution you'd like n/A

Describe alternatives you've considered n/A

Additional context n/A

SamYuan1990 avatar Sep 27 '22 14:09 SamYuan1990

@marceloamaral , could you please add some reason in comments for remove post integration test checking?

SamYuan1990 avatar Sep 27 '22 14:09 SamYuan1990

I'm not saying that post-integration testing isn't useful, there may be some use cases that will be needed

However, for simplicity's sake for now, I think that our integration test should be run before the merge

marceloamaral avatar Sep 28 '22 06:09 marceloamaral

I'm not saying that post-integration testing isn't useful, there may be some use cases that will be needed

However, for simplicity's sake for now, I think that our integration test should be run before the merge

I am going to create a pr for this and let's see.

SamYuan1990 avatar Sep 28 '22 12:09 SamYuan1990

close for now as resolved by #267

SamYuan1990 avatar Oct 22 '22 08:10 SamYuan1990