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 Fourier CLI #2196

Merged
merged 11 commits into from Dec 26, 2020
Merged

Add Fourier CLI #2196

merged 11 commits into from Dec 26, 2020

Conversation

pepicrft
Copy link
Contributor

@pepicrft pepicrft commented Dec 25, 2020

Short description 📝

With the goal of improving the contributor's experience, I'm evolving our already-large Rakefile into a CLI with streamlined commands to interact with the projects and setting up the environment. This PR configures a new Ruby CLI in the repository, bin/fourier (in Ruby), and adds a new GitHub workflow to run its tests. The goal with this effort is also to ensure that our workflows are tested and that work deterministically.

Bear in mind that I'm not moving the current Rake tasks. I'll gradually do that in future PRs.

I'm taking the opportunity to migrate the project to Ruby 3, which was released today

Checklist ✅

  • The code architecture and patterns are consistent with the rest of the codebase.
  • A developer other than the author has verified that the changes work as expected.
  • The changes have been tested following the documented guidelines.
  • The CHANGELOG.md has been updated to reflect the changes. In case of a breaking change, it's been flagged as such.
  • In case the PR introduces changes that affect users, the documentation has been updated.

@pepicrft pepicrft changed the title Add backbone CLI Add Fourier CLI Dec 26, 2020
@pepicrft pepicrft marked this pull request as ready for review December 26, 2020 10:40
@pepicrft pepicrft requested review from a team, natanrolnik and kwridan and removed request for a team December 26, 2020 16:00
@pepicrft pepicrft enabled auto-merge (squash) December 26, 2020 16:36
@pepicrft pepicrft merged commit 80189ef into main Dec 26, 2020
@pepicrft pepicrft deleted the cli branch December 26, 2020 18:03
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.

None yet

1 participant