Details

    Description

      Model it after npm config command. https://docs.npmjs.com/cli/config

      This issue could use a cordova-discuss PR before implementing.

      Might want to migrate telemetry and update-notifier config into this.

      Currently, telemetry and update-notifier use https://github.com/yeoman/configstore under the hood. Stores configs in ~/.config/configstore/

      Attachments

        Activity

          auso Audrey So added a comment - - edited Link to proposal : https://github.com/cordova/cordova-discuss/pull/63/commits/788be4b3a2f360cdc2c5b4ddd46d7547a7f45416
          githubbot ASF GitHub Bot added a comment -

          GitHub user stevengill opened a pull request:

          https://github.com/apache/cordova-cli/pull/276

          CB-11982: rewrote tests for cordova config

          <!--
          Please make sure the checklist boxes are all checked before submitting the PR. The checklist
          is intended as a quick reference, for complete details please see our Contributor Guidelines:

          http://cordova.apache.org/contribute/contribute_guidelines.html

          Thanks!
          -->

              1. Platforms affected
              1. What does this PR do?
              1. What testing has been done on this change?
              1. Checklist
          • [ ] [Reported an issue](http://cordova.apache.org/contribute/issues.html) in the JIRA database
          • [ ] Commit message follows the format: "CB-3232: (android) Fix bug with resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform affected.
          • [ ] Added automated test coverage as appropriate for this change.

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/stevengill/cordova-cli master

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/cordova-cli/pull/276.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #276


          commit dba7c9ab5e80b040da648ac1c6085b5cb8fbf1d6
          Author: Audrey So <audreyso@apache.org>
          Date: 2017-01-31T23:27:34Z

          CB-11982 : added config command that sets, gets, and deletes

          commit 8956c75931f742a773c1e78d49c7dac85fdb6747
          Author: Audrey So <audreyso@apache.org>
          Date: 2017-02-02T00:00:56Z

          CB-11982 : added jasmine tests to test config get, set and delete

          commit c6286f2cc6afd2b89d61087e290284a77da7da09
          Author: Audrey So <audreyso@apache.org>
          Date: 2017-02-02T01:36:39Z

          CB-11982 : updated config command if save, fetch, or browserify is undefined

          commit 7c035ce1ba73f416498d9ee926ac53dcc4888a10
          Author: Audrey So <audreyso@apache.org>
          Date: 2017-02-03T00:39:39Z

          CB-11982 : added edit and ls to delete command

          commit d34e59696d458f80d9a3e72354cefd2eba0199c2
          Author: Audrey So <audreyso@apache.org>
          Date: 2017-03-10T17:26:02Z

          CB-11982 : updated info about new config command

          commit 9053ed00e0bab71a26c3d634e07494ec61696f25
          Author: Audrey So <audreyso@apache.org>
          Date: 2017-03-23T18:13:30Z

          CB-11982 : revised config command wording so it's clearer

          commit 1b5cf978205c221167b10b51e604dc0d72dd3e5d
          Author: Steve Gill <stevengill97@gmail.com>
          Date: 2017-04-19T23:27:32Z

          CB-11982: rewrote cordova config tests


          githubbot ASF GitHub Bot added a comment - GitHub user stevengill opened a pull request: https://github.com/apache/cordova-cli/pull/276 CB-11982 : rewrote tests for cordova config <!-- Please make sure the checklist boxes are all checked before submitting the PR. The checklist is intended as a quick reference, for complete details please see our Contributor Guidelines: http://cordova.apache.org/contribute/contribute_guidelines.html Thanks! --> Platforms affected What does this PR do? What testing has been done on this change? Checklist [ ] [Reported an issue] ( http://cordova.apache.org/contribute/issues.html ) in the JIRA database [ ] Commit message follows the format: "CB-3232: (android) Fix bug with resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform affected. [ ] Added automated test coverage as appropriate for this change. You can merge this pull request into a Git repository by running: $ git pull https://github.com/stevengill/cordova-cli master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cordova-cli/pull/276.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #276 commit dba7c9ab5e80b040da648ac1c6085b5cb8fbf1d6 Author: Audrey So <audreyso@apache.org> Date: 2017-01-31T23:27:34Z CB-11982 : added config command that sets, gets, and deletes commit 8956c75931f742a773c1e78d49c7dac85fdb6747 Author: Audrey So <audreyso@apache.org> Date: 2017-02-02T00:00:56Z CB-11982 : added jasmine tests to test config get, set and delete commit c6286f2cc6afd2b89d61087e290284a77da7da09 Author: Audrey So <audreyso@apache.org> Date: 2017-02-02T01:36:39Z CB-11982 : updated config command if save, fetch, or browserify is undefined commit 7c035ce1ba73f416498d9ee926ac53dcc4888a10 Author: Audrey So <audreyso@apache.org> Date: 2017-02-03T00:39:39Z CB-11982 : added edit and ls to delete command commit d34e59696d458f80d9a3e72354cefd2eba0199c2 Author: Audrey So <audreyso@apache.org> Date: 2017-03-10T17:26:02Z CB-11982 : updated info about new config command commit 9053ed00e0bab71a26c3d634e07494ec61696f25 Author: Audrey So <audreyso@apache.org> Date: 2017-03-23T18:13:30Z CB-11982 : revised config command wording so it's clearer commit 1b5cf978205c221167b10b51e604dc0d72dd3e5d Author: Steve Gill <stevengill97@gmail.com> Date: 2017-04-19T23:27:32Z CB-11982 : rewrote cordova config tests
          githubbot ASF GitHub Bot added a comment -

          Github user stevengill commented on the issue:

          https://github.com/apache/cordova-cli/pull/276

          replaces #267

          githubbot ASF GitHub Bot added a comment - Github user stevengill commented on the issue: https://github.com/apache/cordova-cli/pull/276 replaces #267

          Commit dba7c9ab5e80b040da648ac1c6085b5cb8fbf1d6 in cordova-cli's branch refs/heads/master from Audrey So
          [ https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;h=dba7c9a ]

          CB-11982 : added config command that sets, gets, and deletes

          jira-bot ASF subversion and git services added a comment - Commit dba7c9ab5e80b040da648ac1c6085b5cb8fbf1d6 in cordova-cli's branch refs/heads/master from Audrey So [ https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;h=dba7c9a ] CB-11982 : added config command that sets, gets, and deletes

          Commit 8956c75931f742a773c1e78d49c7dac85fdb6747 in cordova-cli's branch refs/heads/master from Audrey So
          [ https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;h=8956c75 ]

          CB-11982 : added jasmine tests to test config get, set and delete

          jira-bot ASF subversion and git services added a comment - Commit 8956c75931f742a773c1e78d49c7dac85fdb6747 in cordova-cli's branch refs/heads/master from Audrey So [ https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;h=8956c75 ] CB-11982 : added jasmine tests to test config get, set and delete

          Commit c6286f2cc6afd2b89d61087e290284a77da7da09 in cordova-cli's branch refs/heads/master from Audrey So
          [ https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;h=c6286f2 ]

          CB-11982 : updated config command if save, fetch, or browserify is undefined

          jira-bot ASF subversion and git services added a comment - Commit c6286f2cc6afd2b89d61087e290284a77da7da09 in cordova-cli's branch refs/heads/master from Audrey So [ https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;h=c6286f2 ] CB-11982 : updated config command if save, fetch, or browserify is undefined

          Commit 7c035ce1ba73f416498d9ee926ac53dcc4888a10 in cordova-cli's branch refs/heads/master from Audrey So
          [ https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;h=7c035ce ]

          CB-11982 : added edit and ls to delete command

          jira-bot ASF subversion and git services added a comment - Commit 7c035ce1ba73f416498d9ee926ac53dcc4888a10 in cordova-cli's branch refs/heads/master from Audrey So [ https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;h=7c035ce ] CB-11982 : added edit and ls to delete command
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/cordova-cli/pull/276

          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/cordova-cli/pull/276

          Commit d34e59696d458f80d9a3e72354cefd2eba0199c2 in cordova-cli's branch refs/heads/master from Audrey So
          [ https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;h=d34e596 ]

          CB-11982 : updated info about new config command

          jira-bot ASF subversion and git services added a comment - Commit d34e59696d458f80d9a3e72354cefd2eba0199c2 in cordova-cli's branch refs/heads/master from Audrey So [ https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;h=d34e596 ] CB-11982 : updated info about new config command

          Commit 9053ed00e0bab71a26c3d634e07494ec61696f25 in cordova-cli's branch refs/heads/master from Audrey So
          [ https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;h=9053ed0 ]

          CB-11982 : revised config command wording so it's clearer

          jira-bot ASF subversion and git services added a comment - Commit 9053ed00e0bab71a26c3d634e07494ec61696f25 in cordova-cli's branch refs/heads/master from Audrey So [ https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;h=9053ed0 ] CB-11982 : revised config command wording so it's clearer

          Commit 627c98727c141430d9e21a24326572d4b71bd698 in cordova-cli's branch refs/heads/master from Steve Gill
          [ https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;h=627c987 ]

          CB-11982: rewrote cordova config tests

          jira-bot ASF subversion and git services added a comment - Commit 627c98727c141430d9e21a24326572d4b71bd698 in cordova-cli's branch refs/heads/master from Steve Gill [ https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;h=627c987 ] CB-11982 : rewrote cordova config tests

          Commit f1b4dc892ae57be10061eab3df850e03970f26ce in cordova-discuss's branch refs/heads/master from Audrey So
          [ https://gitbox.apache.org/repos/asf?p=cordova-discuss.git;h=f1b4dc8 ]

          GitHub migration proposal (#75)

          • CB-11982 - proposal for new cordova config command
          • cordova-config-command.md removed and pushed up on CB-11982 branch instead
          • github_migration_proposal : proposal to migrate jira issues to github
          jira-bot ASF subversion and git services added a comment - Commit f1b4dc892ae57be10061eab3df850e03970f26ce in cordova-discuss's branch refs/heads/master from Audrey So [ https://gitbox.apache.org/repos/asf?p=cordova-discuss.git;h=f1b4dc8 ] GitHub migration proposal (#75) CB-11982 - proposal for new cordova config command cordova-config-command.md removed and pushed up on CB-11982 branch instead github_migration_proposal : proposal to migrate jira issues to github

          People

            auso Audrey So
            stevegill Steve Gill
            Votes:
            0 Vote for this issue
            Watchers:
            Stop watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment