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 generator #116

Merged
merged 12 commits into from
Jan 4, 2024
Merged

Add generator #116

merged 12 commits into from
Jan 4, 2024

Conversation

ahangarha
Copy link
Contributor

@ahangarha ahangarha commented Nov 22, 2023

⚠️ Note: This PR is on top of PR #110, because it doesn't need to read configuration files. The base branch is deliberately not set to main so that PR review becomes easier.

Tasks:

  • Add generator
    • controlplane.yml
    • Dockerfile
    • templates/gvc.yml
    • templates/rails.yml
    • templates/postgres.yml
  • Add options
    • --destination for custom destination path (defaults to ./.controlplane) - In future if we want to add feature for custom config path.
    • --force to force file change - Canceled in favor of the following task
    • Prevent override on existing configurations
  • Add tests
  • Update Docs
  • Update Changelog

Closes #89

@ahangarha ahangarha force-pushed the dont-read-config-files-for-some-commands branch 2 times, most recently from efa2865 to b88f712 Compare November 29, 2023 11:54
@ahangarha ahangarha changed the base branch from dont-read-config-files-for-some-commands to main December 12, 2023 18:52
@ahangarha ahangarha marked this pull request as ready for review December 12, 2023 18:52
@ahangarha ahangarha changed the title WIP - Add generator Add generator Dec 12, 2023
@ahangarha ahangarha force-pushed the add-generator branch 2 times, most recently from c913f4c to 94acc63 Compare December 20, 2023 16:57
@ahangarha ahangarha changed the title Add generator WIP - Add generator Dec 26, 2023
@ahangarha ahangarha changed the title WIP - Add generator Add generator Dec 29, 2023
@@ -0,0 +1,27 @@
FROM ruby:3.1.2
Copy link
Member

Choose a reason for hiding this comment

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

Why not ruby version from Gemfile?

@borela is there any way that we can have the ruby version in one spot?

Copy link
Member

Choose a reason for hiding this comment

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

Use template for this file.

Copy link
Member

Choose a reason for hiding this comment

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

@@ -0,0 +1,27 @@
FROM ruby:3.1.2
Copy link
Member

Choose a reason for hiding this comment

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

Use template for this file.

@@ -0,0 +1,27 @@
FROM ruby:3.1.2
Copy link
Member

Choose a reason for hiding this comment

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

@justin808 justin808 merged commit 517f254 into main Jan 4, 2024
9 checks passed
@justin808 justin808 deleted the add-generator branch January 4, 2024 07:41
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.

Add generator
2 participants