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

Ignore setting 20190803 #12

Merged
merged 4 commits into from
Aug 3, 2019
Merged

Ignore setting 20190803 #12

merged 4 commits into from
Aug 3, 2019

Conversation

z6wdc
Copy link
Member

@z6wdc z6wdc commented Aug 3, 2019

Edit the ignore setting.
So if we have new format , you can edit the app.yaml.
But please do not put your information in the app.yaml

Create your own secret.yaml in your local and write your own variables to override the setting in the app.yaml
like the following

env_variables:
  CLOUD_SQL_CONNECTION_NAME: your-connection-name
  DB_USER: your-account
  DB_PASS: your-password
  DB_NAME: your-database

@z6wdc z6wdc requested review from bkno21427 and dwi2 August 3, 2019 02:13
@z6wdc z6wdc self-assigned this Aug 3, 2019
Copy link
Contributor

@dwi2 dwi2 left a comment

Choose a reason for hiding this comment

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

LGTM

@z6wdc z6wdc merged commit d451abc into project-yori:master Aug 3, 2019
@z6wdc z6wdc deleted the ignore_setting_20190803 branch August 3, 2019 03:09
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.

2 participants