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

what's the strategy about having this tool in the main organization? #70

Open
moul opened this issue Sep 21, 2024 · 1 comment
Open
Labels
C-question Category: Further information is requested or question

Comments

@moul
Copy link

moul commented Sep 21, 2024

how do we plan to integrate it with the rest of the official tools?

@moul moul added the C-question Category: Further information is requested or question label Sep 21, 2024
@moul moul moved this from 📥 Inbox to Watching in 😎 Manfred's Board Sep 21, 2024
@notJoon
Copy link
Contributor

notJoon commented Sep 23, 2024

how do we plan to integrate it with the rest of the official tools?

@moul

Good and tough question. In fact, since I did not expected to migrate this project to the organization repository from the beginning, I currently lack a strategy.

However I'm considering the following approachs:

  1. Maintain the Lint Tool Separately: Continue managing this tool in a separate repositiory, like this, keeping it independent from the gno codebase as we do currently.

  2. Itegrate as a Submodule: Later, add this tool as a submodule to replace the existing lint implementation and continuously update module periodically, if there are changes.

By applying this method, I believe we can develop and maintain the tool independently of the gno's state. If the integration proceeds smoothly, we might also consider adding it to the CI/CD pipeline. Additionally, this approach os similar to the one used by Rust's Clippy, which I am referencing (although it appears somewhat different from the approach used by go tools).

However, still a lot of stabilization work is needed, it seems best to gather varipus opinions in the meantime and establish a comprehensive plan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-question Category: Further information is requested or question
Projects
None yet
Development

No branches or pull requests

2 participants