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.

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

Add .bazelversion with 5.4.0 #40

Merged
merged 1 commit into from
Jan 5, 2023

Conversation

zerkz
Copy link
Contributor

@zerkz zerkz commented Dec 25, 2022

Adds .bazelversion which points Bazelisk to install bazel 5.4.0.

bazel build ... succeeded for me with 5.4.0 `

  • [ X ] Appropriate changes to README are included in PR

@google-cla
Copy link

google-cla bot commented Dec 25, 2022

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@zerkz
Copy link
Contributor Author

zerkz commented Dec 25, 2022

also -- thank y'all for taking on this project. I read over the discussion over at pmd, and I too, have many of the same pain points you expressed.

I started a side project utility that would lint/warn of Apex Triggers that don't follow a certain pattern (ex. a trigger that doesn't eager load related objects or some other undesired pattern). Shortly after it became apparent that this would be pretty difficult with Jorje API being undocumented/closed.

So, I'm rooting for y'all, and I'll try to help when time/ability allows!

@aaronhurst-google aaronhurst-google merged commit 39ba16e into google:main Jan 5, 2023
@aaronhurst-google
Copy link
Collaborator

Thank you kindly @zerkz for the contribution. Glad to hear that this was useful.

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.

3 participants