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

Introduce suspenders:production:environment generator #1151

Conversation

crackofdusk
Copy link
Contributor

@crackofdusk crackofdusk commented Dec 12, 2023

Creates generator to configuration the production environment. For now,
this means requiring a master key.

Drops configuration for asset_host, since that should be an
infrastructure decision.

In an effort to distinguish between the development environment
configuration in #1149, we use the production namespace.

@crackofdusk crackofdusk mentioned this pull request Dec 12, 2023
17 tasks
@crackofdusk crackofdusk force-pushed the suspenders-3-0-0-production-env-generator branch from 1aaf935 to acb443f Compare December 22, 2023 09:56
@stevepolitodesign
Copy link
Contributor

Noting we might want to set:

config.require_master_key = true

@stevepolitodesign stevepolitodesign force-pushed the suspenders-3-0-0-production-env-generator branch from acb443f to 9bef3a6 Compare April 5, 2024 09:37
@stevepolitodesign stevepolitodesign changed the title Introduce suspenders:production_environment generator Introduce suspenders:production:environment generator Apr 5, 2024
@stevepolitodesign stevepolitodesign marked this pull request as ready for review April 5, 2024 09:38
@stevepolitodesign stevepolitodesign self-assigned this Apr 5, 2024
@stevepolitodesign stevepolitodesign force-pushed the suspenders-3-0-0-production-env-generator branch from 9bef3a6 to c22a177 Compare April 5, 2024 11:29
Creates generator to configuration the production environment. For now,
this means [requiring a master key][master key].

Drops configuration for [asset_host][], since that should be an
infrastructure decision.

In an effort to distinguish between the development environment
configuration in #1149, we use the `production` namespace.

[master key]: https://guides.rubyonrails.org/configuring.html#config-require-master-key
[asset_host]: https://guides.rubyonrails.org/configuring.html#config-asset-host
@stevepolitodesign stevepolitodesign force-pushed the suspenders-3-0-0-production-env-generator branch from c22a177 to 1476279 Compare April 5, 2024 13:59
@stevepolitodesign stevepolitodesign merged commit 667b7ae into suspenders-3-0-0 Apr 5, 2024
2 checks passed
@stevepolitodesign stevepolitodesign deleted the suspenders-3-0-0-production-env-generator branch April 5, 2024 17:32
stevepolitodesign added a commit that referenced this pull request Apr 5, 2024
Follow-up to #1151, #1182, and #1149

Using the `Environments` namespace feels more appropriate, since this
keeps parity with Rails' directory structure.
stevepolitodesign added a commit that referenced this pull request Apr 5, 2024
Follow-up to #1151, #1182, and #1149

Using the `Environments` namespace feels more appropriate, since this
keeps parity with Rails' directory structure.
stevepolitodesign added a commit that referenced this pull request May 10, 2024
Creates generator to configuration the production environment. For now,
this means [requiring a master key][master key].

Drops configuration for [asset_host][], since that should be an
infrastructure decision.

In an effort to distinguish between the development environment
configuration in #1149, we use the `production` namespace.

[master key]: https://guides.rubyonrails.org/configuring.html#config-require-master-key
[asset_host]: https://guides.rubyonrails.org/configuring.html#config-asset-host

Co-authored-by: Steve Polito <[email protected]>
stevepolitodesign added a commit that referenced this pull request May 10, 2024
Follow-up to #1151, #1182, and #1149

Using the `Environments` namespace feels more appropriate, since this
keeps parity with Rails' directory structure.
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.

4 participants