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

Improve build and publish workflow #250

Closed
wants to merge 5 commits into from

Conversation

Elliot-Alexander
Copy link

@Elliot-Alexander Elliot-Alexander commented Jul 25, 2023

Description

Currently just adds commitlint, will add workflow changes soon, been busy with work 🙃

  • Adds commitlint
  • Rewrite workflow to be more segmented with specific jobs (lint, test, build)
  • Has PR specific checks (currently just commitlint but could enforce other tagging, description details etc)
  • Configuration is written to allow for zero change™ full adoption of pnpm workspaces
  • Halves the current CI time

Related Issue

#247

Motivation and Context

#247

How Has This Been Tested?

Screenshots (if appropriate):

Check my current branch to see applicable workflow runs :)

@zernonia
Copy link
Member

zernonia commented Aug 2, 2023

Thanks for this PR @Elliot-Alexander ! Any updates on this? 😁

@Elliot-Alexander Elliot-Alexander force-pushed the add-commitlint branch 4 times, most recently from 674c4a2 to 11ca5f2 Compare August 2, 2023 04:56
@Elliot-Alexander
Copy link
Author

Hey @zernonia, just pushed my first attempt at the CI workflow; have outlined above the changes. I might merge this PR with my other open one re: prettier (which I'll replace with Antfu's eslint) if that sounds alright to you. We can discuss the release workflow on Discord later as will need to coordinate you adding tokens to the repo secrets.

@zernonia
Copy link
Member

zernonia commented Aug 2, 2023

Looking good @Elliot-Alexander ! I think let's not merge this with Prettier. Since that PR shouldn't update any pipeline, and just provide a shared eslint among the packages 😁

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.

2 participants