Skip to content

Update CI workflow and configurations #157

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 27, 2022
Merged

Update CI workflow and configurations #157

merged 1 commit into from
Mar 27, 2022

Conversation

@jamesrweb
Copy link
Member

@jamesrweb jamesrweb commented Mar 27, 2022

Proposed Changes

  • Update the CI workflow to handle formatting just the same as we do for linting
  • Rename the CI workflow file to reflect the configuration name more correctly
  • Update the rollup configuration to more dynamically generate the bundle configurations
  • Create new npm scripts for a local ci reproduction case and for checking formatting rules are being applied correctly

@jamesrweb jamesrweb added enhancement dependencies Pull requests that update a dependency file labels Mar 27, 2022
@jamesrweb jamesrweb requested a review from yevdyko March 27, 2022 20:54
@jamesrweb jamesrweb self-assigned this Mar 27, 2022
@jamesrweb jamesrweb enabled auto-merge (squash) March 27, 2022 20:55
yevdyko
yevdyko approved these changes Mar 27, 2022
View reviewed changes
Copy link
Contributor

@yevdyko yevdyko left a comment

Choose a reason for hiding this comment

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

Thanks @jamesrweb, this looks good to me! 🚢 🚢 🚢

@jamesrweb jamesrweb merged commit f6eb46e into master Mar 27, 2022
@jamesrweb jamesrweb deleted the update-ci-workflow-config branch March 27, 2022 22:08
jamesrweb added a commit that referenced this pull request Aug 15, 2022
@jamesrweb
Update CI configuration and configurations (#157)
1213f73
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yevdyko yevdyko yevdyko approved these changes

Assignees

@jamesrweb jamesrweb

Labels

dependencies Pull requests that update a dependency file enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jamesrweb @yevdyko