Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(e2e): run uptest every pr in this repo #53

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

haarchri
Copy link
Contributor

@haarchri haarchri commented Jul 31, 2023

Description of your changes

Fixes #34

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Signed-off-by: Christopher Haar <christopher.haar@upbound.io>
Copy link
Member

@sergenyalcin sergenyalcin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @haarchri I left a few comments.


jobs:
uptest:
if: ${{ github.event.issue.pull_request }}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to add a on block to the workflow definition similar to this? Or will this definition be sufficient to run workflow?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not we use re-usable workflow approach for here? Are there steps we don't want to use?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a test as a required status check that makes sure the generated provider is running
2 participants