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

Yaml parser support for flow style mappings #423

Closed
gadams00 opened this issue Apr 8, 2021 · 1 comment · Fixed by #1721
Closed

Yaml parser support for flow style mappings #423

gadams00 opened this issue Apr 8, 2021 · 1 comment · Fixed by #1721
Assignees
@traceyyoshima
Labels
enhancement New feature or request
Milestone
7.23.0

Comments

@gadams00
Copy link
Contributor

gadams00 commented Apr 8, 2021

The current yaml parser (and printer) makes the assumption that parsed yaml documents contain only block style elements. Add support for Yaml flow style sequences and mappings, and any other flow style yaml document elements.

example flow style sequence:

a: [foo bar, baz]

example flow style mapping:

b: {c: 3, d: 4}
@sambsnyd
Copy link
Member

flow-styled sequences are now handled as of #573.
flow-styled mappings are, as of yet, unsupported.

@sambsnyd sambsnyd added the enhancement New feature or request label May 25, 2021
@tkvangorder tkvangorder changed the title Yaml parser support for flow style Yaml parser support for flow style mappings Apr 8, 2022
@traceyyoshima traceyyoshima mentioned this issue Apr 12, 2022
Closed
@traceyyoshima traceyyoshima self-assigned this Apr 26, 2022
traceyyoshima added a commit that referenced this issue Apr 26, 2022
@traceyyoshima
Added support for flow style mappings to Yaml.
52e0a41 
fixes #423
@traceyyoshima traceyyoshima mentioned this issue Apr 26, 2022
Merged
traceyyoshima added a commit that referenced this issue Apr 26, 2022
@traceyyoshima
Added support for flow style mappings to Yaml.
230fd75 
fixes #423
traceyyoshima added a commit that referenced this issue Apr 28, 2022
@traceyyoshima
Added support for flow style mappings to Yaml.
a4a457a 
fixes #423
traceyyoshima added a commit that referenced this issue Apr 28, 2022
@traceyyoshima
Added support for flow style mappings to Yaml. (#1721)
bce25ef 
fixes #423
@tkvangorder tkvangorder added this to the 7.23.0 milestone May 10, 2022
yeikel pushed a commit to yeikel/rewrite that referenced this issue Jan 15, 2023
@dependabot
chore(ci): bump rocksdbjni from 7.5.3 to 7.6.0 (openrewrite#423)
dc4d13f 
Bumps [rocksdbjni](https://github.com/facebook/rocksdb) from 7.5.3 to 7.6.0.
- [Release notes](https://github.com/facebook/rocksdb/releases)
- [Changelog](https://github.com/facebook/rocksdb/blob/main/HISTORY.md)
- [Commits](facebook/rocksdb@v7.5.3...v7.6.0)

---
updated-dependencies:
- dependency-name: org.rocksdb:rocksdbjni
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@traceyyoshima traceyyoshima

Labels
enhancement New feature or request
Projects
OpenRewrite
Archived in project
Milestone
7.23.0
Development

Successfully merging a pull request may close this issue.

Added support for flow style mappings to Yaml. openrewrite/rewrite
4 participants
@tkvangorder @gadams00 @sambsnyd @traceyyoshima