Skip to content

add go linter to linux workflow - #3

Open
singholt wants to merge 2 commits into
masterfrom
go-linter
Open

add go linter to linux workflow#3
singholt wants to merge 2 commits into
masterfrom
go-linter

Conversation

@singholt

Copy link
Copy Markdown
Owner

Summary

testing go-linter

Implementation details

Testing

New tests cover the changes:

Description for the changelog

Licensing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@singholt
singholt force-pushed the go-linter branch 12 times, most recently from 93c1c17 to d9c012d Compare June 22, 2022 19:06
singholt pushed a commit that referenced this pull request Aug 15, 2022
…ainers. (#3)

* First cut of the changes.

* Intermediate check-in before refactoring to hashing the paths.

* Refactored the code to use a concatenated hashed value for the CredSpec filename.

* Cleaning the cruft.

* Simplified the code a bit and incorporated the first round of PR review comments.

* Arranged the imports as per the language spec.

* Fixed a broken test.

* Fixed a few more broken tests.

* Refactored the code to only use the credspecContainer Map.

* Fixed more references and tests.

* Fixed more tests.

* Fixed more tests.

* Fixed more tests.

* Fixed more tests.

* Improved the documentation for the member variable.

* Incorporated some review comments.

* Removed the unnecessary method and cleaned up the code a bit.

* Some more clean-up of the code.

* Tidying the imports.
singholt pushed a commit that referenced this pull request Jan 2, 2025
* Added the first cut of the implementation for the CSIDriver.

* Cleaned the branch with unneeded vendor files.

---------

Co-authored-by: Saurabh Chakravarty <drsaurab@amazon.com>
@singholt
singholt force-pushed the master branch 4 times, most recently from 352d9fd to cbdf701 Compare March 10, 2026 00:07
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.

1 participant