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

Opensource - gitignore update #778

Closed
wants to merge 3 commits into from
Closed

Opensource - gitignore update #778

wants to merge 3 commits into from

Conversation

calvinhlee23
Copy link
Contributor

  • going forward, the following changes will ignore listed files / folders.

@@ -13,6 +13,12 @@
/dist
/.next

# deploy & configs
/config/constants.js
Copy link
Contributor

Choose a reason for hiding this comment

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

instead of removing this constants file, maybe we should have a constants_local or something else?

I think the app will break without this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will do. Thanks

@lightninglu10 lightninglu10 deleted the opensource branch December 9, 2021 17:59
@lightninglu10 lightninglu10 restored the opensource branch December 9, 2021 17:59
@lightninglu10 lightninglu10 deleted the opensource branch March 10, 2022 21:43
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