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

Fix update bugs #53

Merged
merged 4 commits into from
Apr 22, 2023
Merged

Fix update bugs #53

merged 4 commits into from
Apr 22, 2023

Conversation

taechae
Copy link
Contributor

@taechae taechae commented Apr 22, 2023

Not sure if your last change to update existing occurrences ever worked. Fixed several related issues.

@taechae taechae requested a review from yeshwanth1993 April 22, 2023 01:25
@taechae
Copy link
Contributor Author

taechae commented Apr 22, 2023

Also, it looks like PackageIssues are not merged. Was that already implemented?

yeshwanth1993
yeshwanth1993 previously approved these changes Apr 22, 2023
@taechae
Copy link
Contributor Author

taechae commented Apr 22, 2023

Did you ever figure out why the go linter is not working? It doesn't seem to work on my test machine either.

@yeshwanth1993
Copy link
Collaborator

Did you ever figure out why the go linter is not working? It doesn't seem to work on my test machine either.

It worked after I changed it to v3 from referencing a digest.

@yeshwanth1993 yeshwanth1993 self-requested a review April 22, 2023 01:47
@taechae taechae merged commit 7ee1fb4 into main Apr 22, 2023
@taechae taechae deleted the taechae-fixupdate branch April 22, 2023 01:55
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