Skip to content

Commit

Permalink
fix: Prevent setting a constant variable
Browse files Browse the repository at this point in the history
It appears that there was an option --patterns in the past, whcih did not work in the fork?
  • Loading branch information
prantlf committed Mar 7, 2023
1 parent 02a5789 commit c7e940c
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 8 deletions.
6 changes: 1 addition & 5 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,11 +8,7 @@

### BREAKING CHANGES

* Although you shouldn't notice any change on the behaviour of the command line, something unexpected might've changed. Something did change
- if you're annoyed by inserting "--" between the multi-value option and other
arguments, you don't have to do it any more. Multi-value options can be entered
either using the option prefix multiple times for each value, or using
the option prefix just once and separating the values by commas.
* Although you shouldn't notice any change on the behaviour of the command line, something unexpected might've changed. Something did change: if you're annoyed by inserting "--" between the multi-value option and other arguments, you don't have to do it any more. Multi-value options can be entered either using the option prefix multiple times for each value, or using the option prefix just once and separating the values by commas.

# [13.1.0](https://github.com/prantlf/jsonlint/compare/v13.0.1...v13.1.0) (2023-03-05)

Expand Down
3 changes: 0 additions & 3 deletions lib/cli.js
Original file line number Diff line number Diff line change
Expand Up @@ -499,9 +499,6 @@ function processPatterns (patterns) {
}

function main () {
if (!args.length) {
args = params.patterns || []
}
if (args.length) {
const dynamic = args.some(file => isDynamicPattern(file))
if (dynamic) {
Expand Down

1 comment on commit c7e940c

@prantlf
Copy link
Owner Author

@prantlf prantlf commented on c7e940c Mar 7, 2023

Choose a reason for hiding this comment

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

Fixes #17.

Please sign in to comment.