Skip to content
This repository has been archived by the owner on Jun 28, 2022. It is now read-only.

Fix string literal character escaping in Ruby generator #2861

Merged
merged 2 commits into from
Jun 28, 2019

Conversation

vam-google
Copy link
Contributor

@vam-google vam-google commented Jun 28, 2019

This seems like missing part of #2782.

Note, the left part of the diff (i.e. file before this PR) for ruby_library.baseline file contains non-printable whitespaces.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 28, 2019
Copy link
Contributor

@noahdietz noahdietz left a comment

Choose a reason for hiding this comment

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

lgtm

@vam-google vam-google requested a review from dazuma June 28, 2019 23:15
@codecov
Copy link

codecov bot commented Jun 28, 2019

Codecov Report

Merging #2861 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2861      +/-   ##
============================================
- Coverage     87.16%   87.16%   -0.01%     
  Complexity     5791     5791              
============================================
  Files           476      476              
  Lines         22943    22944       +1     
  Branches       2475     2476       +1     
============================================
  Hits          19998    19998              
  Misses         2111     2111              
- Partials        834      835       +1
Impacted Files Coverage Δ Complexity Δ
...n/transformer/ruby/RubyModelTypeNameConverter.java 86.84% <100%> (ø) 18 <0> (ø) ⬇️
...om/google/api/codegen/gapic/GapicGeneratorApp.java 54.95% <0%> (-0.5%) 6% <0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5daa523...9b7a7a5. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented Jun 28, 2019

Codecov Report

Merging #2861 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2861      +/-   ##
============================================
- Coverage     87.16%   87.16%   -0.01%     
  Complexity     5791     5791              
============================================
  Files           476      476              
  Lines         22943    22944       +1     
  Branches       2475     2476       +1     
============================================
  Hits          19998    19998              
  Misses         2111     2111              
- Partials        834      835       +1
Impacted Files Coverage Δ Complexity Δ
...n/transformer/ruby/RubyModelTypeNameConverter.java 86.84% <100%> (ø) 18 <0> (ø) ⬇️
...om/google/api/codegen/gapic/GapicGeneratorApp.java 54.95% <0%> (-0.5%) 6% <0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5daa523...9b7a7a5. Read the comment docs.

@vam-google vam-google merged commit c04c006 into googleapis:master Jun 28, 2019
alexander-fenster added a commit to googleapis/artman that referenced this pull request Jul 1, 2019
Fixes included:

- Allow samplegen without sample configs googleapis/gapic-generator#2860
- Fix string literal character escaping in Ruby generator googleapis/gapic-generator#2861
alexander-fenster added a commit to googleapis/artman that referenced this pull request Jul 1, 2019
Fixes included:

- Allow samplegen without sample configs googleapis/gapic-generator#2860
- Fix string literal character escaping in Ruby generator googleapis/gapic-generator#2861
alexander-fenster added a commit to googleapis/artman that referenced this pull request Jul 1, 2019
In this release:

#653 is fixed by updating gapic-generator.

gapic-generator changes included:

- Allow samplegen without sample configs googleapis/gapic-generator#2860
- Fix string literal character escaping in Ruby generator googleapis/gapic-generator#2861
alexander-fenster added a commit to googleapis/artman that referenced this pull request Jul 1, 2019
In this release:

#653 is fixed by updating gapic-generator.

gapic-generator changes included:

- Allow samplegen without sample configs googleapis/gapic-generator#2860
- Fix string literal character escaping in Ruby generator googleapis/gapic-generator#2861
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants