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

Add CI for ayab-esp32 #34

Merged
merged 6 commits into from
Mar 1, 2024
Merged

Add CI for ayab-esp32 #34

merged 6 commits into from
Mar 1, 2024

Conversation

VIPQualityPost
Copy link
Contributor

No description provided.

@VIPQualityPost
Copy link
Contributor Author

VIPQualityPost commented Feb 28, 2024

This adds:

  • On any PR involving ayab-esp32 directory, it does ERC and DRC checks and makes sure they are clean
  • On any PR involving ayab-esp32 directory that gets merged into main, assembly documentation is generated and saved as an artifact from the action. Unclear to me at this point the best way to keep the artifact as these have a limited life of 1 month or something. Possible to self-create another PR with [no ci] flag so that those files can get pushed into repo but I did not try to do that yet.

@t0mpr1c3
Copy link
Contributor

t0mpr1c3 commented Feb 29, 2024

If it's a tagged version, you could upload the artifact to the release, otherwise 1 month is probably fine. I think it might be 90 days actually.

@VIPQualityPost
Copy link
Contributor Author

VIPQualityPost commented Feb 29, 2024

Yea, I was thinking more on it last night, and I think that generating the artifact on each merge to main is good, and then when we actually cut a hardware release (i.e. buy boards) then we can just take the artifact and upload to a release makes sense, at least for all the production files as we should not circulate un-verified files.
The only question is if we want the auto-generated schematics / layout PDFs to live as part of the repo so that you can peruse "current state" of project easily without needing to download software. This could always come in as a follow up PR with [skip ci] tag although that is a bit manual to remember to do that. Or I can add a step to the git action to open that PR immediately actually.

@VIPQualityPost
Copy link
Contributor Author

OK, this should work, we just need to create a secret github token with repo access. It will open a PR from ayab-hardware/documentation into main, skipping CI to add the generated PDFs to ayab-hardware/ayab-esp32/pdf/

@t0mpr1c3
Copy link
Contributor

t0mpr1c3 commented Feb 29, 2024

We tend not to keep other auto-generated files, but I think we should probably keep schematic pdfs and gerbers.

@VIPQualityPost
Copy link
Contributor Author

Yes, I set it to only add the PDFs. The "real" production files needed to order are still kept as artifacts so they can be used in a tagged release.
@dl1com can you make the secret GITHUB_TOKEN for this?

@VIPQualityPost
Copy link
Contributor Author

OH, I misunderstood, the token is generated as part of the action, so it is already pre-verified. So this should be good to go.

@dl1com dl1com merged commit 353bec1 into main Mar 1, 2024
1 check passed
@dl1com dl1com deleted the VIPQualityPost-patch-1 branch March 1, 2024 08:43
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.

3 participants