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

Cleanup Gemfile groups #4537

Merged
merged 5 commits into from
Sep 6, 2022

Conversation

elia
Copy link
Member

@elia elia commented Sep 1, 2022

Summary

The :frontend group was still present in the Gemfile, also we can skip a few gems in the CI.

Checklist

Check out our PR guidelines for more details.

The following are mandatory for all PRs:

  • I have written a thorough PR description.
  • I have kept my commits small and atomic.
  • I have used clear, explanatory commit messages.

The following are not always needed (cross them out if they are not):

  • I have added automated tests to cover my changes.
  • I have attached screenshots to demo visual changes.
  • I have opened a PR to update the guides.
  • I have updated the readme to account for my changes.

elia added 5 commits September 1, 2022 10:32
Moved the 'i18n-tasks' gem to top-level as it's being used by core and
api specs.
We're not testing against it, can be revisited if we'll add it back
to the CI.
`fast_sqlite` already depends on `sqlite3`.
@elia elia force-pushed the elia/gemfile-frontend-group branch from 7422672 to 417937f Compare September 1, 2022 13:36
@github-actions github-actions bot added the changelog:solidus_core Changes to the solidus_core gem label Sep 1, 2022
@elia elia marked this pull request as ready for review September 2, 2022 09:42
@waiting-for-dev waiting-for-dev added the type:enhancement Proposed or newly added feature label Sep 5, 2022
Copy link
Member

@kennyadsl kennyadsl left a comment

Choose a reason for hiding this comment

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

Thanks Elia, all nice optimiziations. May I suggest to change the title of the PR? It's only related to one of the commits, maybe something more generic will be more appropriate.

@elia elia changed the title Remove the frontend group from the Gemfile Cleanup Gemfile groups Sep 6, 2022
Copy link
Member

@spaghetticode spaghetticode left a comment

Choose a reason for hiding this comment

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

Thanks @elia !

@kennyadsl kennyadsl merged commit a5534a5 into solidusio:master Sep 6, 2022
@kennyadsl kennyadsl deleted the elia/gemfile-frontend-group branch September 6, 2022 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:solidus_core Changes to the solidus_core gem type:enhancement Proposed or newly added feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants