Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

rework sample for dev hub #1

Merged
merged 19 commits into from
Jul 11, 2023
Merged

rework sample for dev hub #1

merged 19 commits into from
Jul 11, 2023

Conversation

baermat
Copy link
Collaborator

@baermat baermat commented Jun 23, 2023

This PR adapts the sample to be ready for the developer hub, according to our standards. Work includes

  • changed diagram
  • added Licensing
  • added Contributing section
  • overhauled README, adding step by step instructions to execute the sample
  • added action for automatically running the sample, executing a smoke test and pushing notification to slack in case of an error
  • added webhook url
  • renamed sample
  • grant access to core team

@baermat baermat requested a review from steffyP June 27, 2023 07:55
@baermat baermat marked this pull request as ready for review June 27, 2023 07:55
Copy link

@steffyP steffyP left a comment

Choose a reason for hiding this comment

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

Nice work 🎉
From my side, everything looks fine (except a small typo) 👍
Please also add "topics" to the project description so it can be found easier 🙏 (guidelines are in notion).

Great to have the CI pipeline, with a smoke test as well 🙂
We might want to add the keepalive-workflow so the workflow will not go stale.
Also tagging @HarshCasper for the final approval 🙏

uses: actions/upload-artifact@v3
with:
name: diagnose.json.gz
path: ./diagnose.json.gz
Copy link

Choose a reason for hiding this comment

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

we haven't really decided yet on how we want to prevent the workflows from getting stale.
I started adding the keepalive-workflow action for the workflows where we test cloudpods, but I don't think we will be able those workflows to all samples in the first iteration.
Therefore, it might be a good idea to add the action in this main.yml for the specific sample? WDYT @HarshCasper?

@steffyP steffyP requested a review from HarshCasper June 27, 2023 10:31
@HarshCasper HarshCasper merged commit 7606436 into main Jul 11, 2023
@steffyP steffyP deleted the developer-hub branch July 11, 2023 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants