Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

ddixit14
Copy link
Contributor

@ddixit14 ddixit14 commented Jan 6, 2023

No description provided.

@ddixit14 ddixit14 requested a review from a team as a code owner January 6, 2023 17:45
WORKSPACE Outdated
repositories = ["https://repo.maven.apache.org/maven2/"],
)

<!--- {x-version-update-start:gapic-generator-java:released} -->
Copy link
Collaborator

Choose a reason for hiding this comment

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

This comment style is for xmls only and it may not work for WORKSPACE

Copy link
Contributor Author

@ddixit14 ddixit14 Jan 6, 2023

Choose a reason for hiding this comment

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

It works for a README.md file, so let's see if it works here as well. And this annotation is not for inline version updates like we do in xml files, this is a block update.

Copy link
Contributor Author

@ddixit14 ddixit14 Jan 6, 2023

Choose a reason for hiding this comment

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

The build failed with syntax error.

(17:48:47) ERROR: /home/runner/work/gapic-generator-java/gapic-generator-java/WORKSPACE:67:1: syntax error at '<': expected expression
(17:48:47) ERROR: /home/runner/work/gapic-generator-java/gapic-generator-java/WORKSPACE:67:2: invalid character: '!'
(17:48:47) ERROR: /home/runner/work/gapic-generator-java/gapic-generator-java/WORKSPACE:6[9](https://github.com/googleapis/gapic-generator-java/actions/runs/3857289723/jobs/6574513783#step:7:10):2: invalid character: '!'

I've reverted it back to previous annotations. Let's see if these annotations work with specifying "extraFiles" in the yaml config.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@ddixit14 ddixit14 merged commit 33dd784 into main Jan 6, 2023
@ddixit14 ddixit14 deleted the rp-fix branch January 6, 2023 18:36
@ddixit14 ddixit14 added the release-please:force-run To run release-please label Jan 6, 2023
@release-please release-please bot removed the release-please:force-run To run release-please label Jan 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants