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

persist a configuration on first run #353

Merged
merged 3 commits into from
Sep 19, 2022
Merged

persist a configuration on first run #353

merged 3 commits into from
Sep 19, 2022

Conversation

miraclx
Copy link
Owner

@miraclx miraclx commented Sep 19, 2022

Stores relevant configuration data in the user specific configuration file, allowing users directly modify that.

Now, the precedence is:

  • CLI options (highest-precedence)
  • --config <FILE>, if present
  • User specific configuration, if present
  • conf.json(default fallback)

@github-actions
Copy link

github-actions bot commented Sep 19, 2022


🐋 🤖

A docker image for this PR has been built!

docker pull freyrcli/freyrjs-git:pr-353
Base (master)
This Patch


What's this?

This docker image is a self-contained sandbox that includes all the patches made in this PR. Allowing others to easily use your patches without waiting for it to get merged and released officially.

For more context, see https://github.com/miraclx/freyr-js#docker-development.

@miraclx miraclx merged commit 98da67f into master Sep 19, 2022
@miraclx miraclx deleted the persist-config branch September 19, 2022 03:22
@miraclx miraclx mentioned this pull request Dec 17, 2022
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.

1 participant