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.

Sign up for GitHub

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

Jump to bottom

Explaination of 'operator_position' is absent from README.md #1047

Closed
abbotto opened this issue Oct 27, 2016 · 4 comments · Fixed by #1089
Closed

Explaination of 'operator_position' is absent from README.md #1047

abbotto opened this issue Oct 27, 2016 · 4 comments · Fixed by #1089
Labels
type: bug type: task
Milestone
v1.6.9

Comments

@abbotto
Copy link

abbotto commented Oct 27, 2016 •
edited
Loading

Description
@abbotto abbotto changed the title Explaination of operator_position is absent from README.md Explaination of 'operator_position' is absent from README.md Oct 27, 2016
@bitwiseman bitwiseman added this to the v1.7.x milestone Dec 26, 2016
@bitwiseman
Copy link
Member

@abbotto or @olsonpm - Could you provide a PR with the update?

@olsonpm
Copy link
Contributor

olsonpm commented Dec 26, 2016

@bitwiseman for sure. Give me a couple days to find some time. I'm knee deep in some react code atm.

@bitwiseman
Copy link
Member

@olsonpm - Thanks! No hurry.

@olsonpm
Copy link
Contributor

olsonpm commented Dec 29, 2016

Ah - this is why I left it out in the readme:

#735 (comment)

I looked at the readme and couldn't figure out why some cli options were left out so I'll leave it be for now.

Sorry for not putting a reminder to get that in prior to the merge @abbotto . I appreciate the issue because I do try to prioritize documentation.

I'll have a PR soon

olsonpm pushed a commit to olsonpm/js-beautify that referenced this issue Dec 29, 2016
Update js-beautify CLI options in README
05dbf2e 
fixes beautifier#1047

 - update cli options with exact output from `$ js-beautify --help`
 - add missing json defaults found in the CLI
 - reorder json defaults to correspond with CLI output
 - remove 'keep_function_indentation' from documentation since it is unused
 - remove html beautifier specific defaults (updating that section is out of
   scope for this commit)
 - separate defaults exposed by the cli and those which aren't
 - reword the compatibility notice as a result of the default separation
@olsonpm olsonpm mentioned this issue Dec 29, 2016
Merged
olsonpm pushed a commit to olsonpm/js-beautify that referenced this issue Dec 30, 2016
Update js-beautify CLI options in README
a48ea78 
fixes beautifier#1047

 - update cli options with exact output from `$ js-beautify --help`
 - add missing json defaults found in the CLI
 - reorder json defaults to correspond with CLI output
 - remove 'keep_function_indentation' from documentation since it is unused
 - remove html beautifier specific defaults (updating that section is out of
   scope for this commit)
 - separate defaults exposed by the cli and those which aren't
 - reword the compatibility notice as a result of the default separation
olsonpm pushed a commit to olsonpm/js-beautify that referenced this issue Jan 2, 2017
Update js-beautify CLI options in README
07828b8 
fixes beautifier#1047

 - update cli options with exact output from `$ js-beautify --help`
 - even out indentation under CLI "Beautifier Options"
 - add missing json defaults found in the CLI
 - reorder json defaults to correspond with CLI output
 - remove 'keep_function_indentation' from documentation since it is unused
 - remove html beautifier specific defaults (updating that section is out of
   scope for this commit)
 - separate defaults exposed by the cli and those which aren't
 - reword the compatibility notice as a result of the default separation
olsonpm pushed a commit to olsonpm/js-beautify that referenced this issue Jan 2, 2017
Update js-beautify CLI options in README
90037b4 
fixes beautifier#1047

 - update cli options with exact output from `$ js-beautify --help`
 - even out indentation under CLI "Beautifier Options"
 - add missing json defaults found in the CLI
 - reorder json defaults to correspond with CLI output
 - remove 'keep_function_indentation' from documentation since it is unused
 - remove html beautifier specific defaults (updating that section is out of
   scope for this commit)
 - separate defaults exposed by the cli and those which aren't
 - reword the compatibility notice as a result of the default separation
olsonpm pushed a commit to olsonpm/js-beautify that referenced this issue Jan 2, 2017
Update js-beautify CLI options in README
e649dc1 
fixes beautifier#1047

 - update cli options with exact output from `$ js-beautify --help`
 - even out indentation under CLI "Beautifier Options"
 - add missing json defaults found in the CLI
 - reorder json defaults to correspond with CLI output
 - remove 'keep_function_indentation' from documentation since it is unused
 - remove html beautifier specific defaults (updating that section is out of
   scope for this commit)
 - separate defaults exposed by the cli and those which aren't
 - reword the compatibility notice as a result of the default separation
olsonpm pushed a commit to olsonpm/js-beautify that referenced this issue Jan 2, 2017
Update js-beautify CLI options in README
2dcb0fa 
fixes beautifier#1047

 - update cli options with exact output from `$ js-beautify --help`
 - even out indentation under CLI "Beautifier Options"
 - add missing json defaults found in the CLI
 - reorder json defaults to correspond with CLI output
 - remove 'keep_function_indentation' from documentation since it is unused
 - remove html beautifier specific defaults (updating that section is out of
   scope for this commit)
 - separate defaults exposed by the cli and those which aren't
 - reword the compatibility notice as a result of the default separation
bitwiseman pushed a commit that referenced this issue Jan 3, 2017
@olsonpm @bitwiseman
Update js-beautify CLI options in README (#1089)
1546edf 
fixes #1047

 - update cli options with exact output from `$ js-beautify --help`
 - even out indentation under CLI "Beautifier Options"
 - add missing json defaults found in the CLI
 - reorder json defaults to correspond with CLI output
 - remove 'keep_function_indentation' from documentation since it is unused
 - remove html beautifier specific defaults (updating that section is out of
   scope for this commit)
 - separate defaults exposed by the cli and those which aren't
 - reword the compatibility notice as a result of the default separation
@bitwiseman bitwiseman modified the milestones: v1.7.x, v1.6.9 Feb 11, 2017
@pyup-bot pyup-bot mentioned this issue Mar 5, 2017
Merged
0-wiz-0 pushed a commit to NetBSD/pkgsrc-wip that referenced this issue Apr 14, 2017
@iamleot
py-jsbeautifier: Update wip/py-jsbeautifier to 1.6.12
99e2edb 
pkgsrc changes:
 - Use ALTERNATIVES in order to permit multi-pkgs to coexists and adjust
   PLIST and introduce Makefile post-install target accordingly

Changes:
* CSS: Preserve Newlines ([#537](beautifier/js-beautify#537))

Reverted #1117 - Preserve newlines broken

* On beautify, new line before next CSS selector ([#1142](beautifier/js-beautify#1142))

Added `preserver_newlines` to css beautifier

* Fixed html formatting issue with attribute wrap (Thanks, @HookyQR!)
* Fixed python package publishing

* Wrong HTML beautification starting with v1.6.5 ([#1115](beautifier/js-beautify#1115))
* Ignore linebreak when meet handlebar ([#1104](beautifier/js-beautify#1104))
* Lines are not un-indented correctly when attributes are wrapped ([#1103](beautifier/js-beautify#1103))
* force-aligned is not aligned when indenting with tabs ([#1102](beautifier/js-beautify#1102))
* Python package fails to publish  ([#1101](beautifier/js-beautify#1101))
* Explaination of 'operator_position' is absent from README.md ([#1047](beautifier/js-beautify#1047))

* Fixed a batch of comment and semicolon-less code bugs

* Incorrect indentation after loop with comment ([#1090](beautifier/js-beautify#1090))
* Extra newline is inserted after beautifying code with anonymous function ([#1085](beautifier/js-beautify#1085))
* end brace with next comment line make bad indent ([#1043](beautifier/js-beautify#1043))
* Javascript comment in last line doesn't beautify well ([#964](beautifier/js-beautify#964))
* indent doesn't work with comment (jsdoc) ([#913](beautifier/js-beautify#913))
* Wrong indentation, when new line between chained methods ([#892](beautifier/js-beautify#892))
* Comments in a non-semicolon style have extra indent ([#815](beautifier/js-beautify#815))
* [bug] Incorrect indentation due to commented line(s) following a function call with a function argument. ([#713](beautifier/js-beautify#713))
* Wrong indent formatting ([#569](beautifier/js-beautify#569))

Added `content_unformatted` option (Thanks @arai-a)

* HTML pre code indentation ([#928](beautifier/js-beautify#928))
* Beautify script/style tags but ignore their inner JS/CSS content ([#906](beautifier/js-beautify#906))

* Added support for editorconfig from stdin
* Added js-beautify to cdnjs
* Fixed CRLF to LF for HTML and CSS on windows
* Added inheritance/overriding to config format (Thanks @DaniGuardiola and @HookyQR)
* Added `force-align` to `wrap-attributes` (Thanks @LukinoS)
* Added `force-expand-multiline` to `wrap-attributes` (Thanks @tobias-zucali)
* Added `preserve-inline` as independent brace setting (Thanks @Coburn37)
* Fixed handlebars with angle-braces (Thanks @mmsqe)

* Wrong indentation for comment after nested unbraced control constructs ([#1079](beautifier/js-beautify#1079))
* Should prefer breaking the line after operator ? instead of before operator < ([#1073](beautifier/js-beautify#1073))
* New option "force-expand-multiline" for "wrap_attributes" ([#1070](beautifier/js-beautify#1070))
* Breaks if html file starts with comment ([#1068](beautifier/js-beautify#1068))
* collapse-preserve-inline restricts users to collapse brace_style ([#1057](beautifier/js-beautify#1057))
* Parsing failure on numbers with "e" ([#1054](beautifier/js-beautify#1054))
* Issue with Browser Instructions ([#1053](beautifier/js-beautify#1053))
* Add preserve inline function for expand style braces ([#1052](beautifier/js-beautify#1052))
* Update years in LICENSE ([#1038](beautifier/js-beautify#1038))
* JS. Switch with template literals. Unexpected indentation. ([#1030](beautifier/js-beautify#1030))
* The object with spread object formatted not correctly ([#1023](beautifier/js-beautify#1023))
* Bad output generator function in class ([#1013](beautifier/js-beautify#1013))
* Support editorconfig for stdin ([#1012](beautifier/js-beautify#1012))
* Publish to cdnjs ([#992](beautifier/js-beautify#992))
* breaks if handlebars comments contain handlebars tags ([#930](beautifier/js-beautify#930))
* Using jsbeautifyrc is broken ([#929](beautifier/js-beautify#929))
* Option to put HTML attributes on their own lines, aligned ([#916](beautifier/js-beautify#916))
* Erroneously changes CRLF to LF on Windows in HTML and CSS ([#899](beautifier/js-beautify#899))
* Weird space in {get } vs { normal } ([#888](beautifier/js-beautify#888))
* Bad for-of formatting with constant Array ([#875](beautifier/js-beautify#875))
* Problems with filter property in css and scss ([#755](beautifier/js-beautify#755))
* Add "collapse-one-line" option for non-collapse brace styles  ([#487](beautifier/js-beautify#487))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: bug type: task
Projects
None yet
Milestone
v1.6.9
Development

Successfully merging a pull request may close this issue.

Update js-beautify CLI options in README olsonpm/js-beautify
3 participants
@abbotto @bitwiseman @olsonpm