Conversation
* Remove VSCode config and documentation files Deleted all files in the .vscode directory and several documentation markdown files. This cleans up editor-specific settings and outdated or unnecessary documentation from the repository. * Add linter directives and project dictionary Added a custom dictionary for the word 'noctx' in the project settings. Suppressed 'noctx' and 'revive' linter warnings in core.go and api.go, respectively, to address linter feedback and document intentional code decisions. * Add comments for future context and linter exceptions Added explanatory comments regarding future use of context in core.go and clarified nolint:revive usage in api.go and parse.go, indicating that package naming will not be changed for now. * Update comments and dictionary entries for clarity Added 'cunyue' to the project dictionary and updated inline comments in Go files to use proper comment syntax for clarity and consistency. * Remove nolint directive from api package declaration Deleted the 'nolint:revive' comment from the package declaration in api.go, allowing standard linting checks to apply. * Remove unused api.go and update parse.go package comment Deleted the core/internal/api/api.go file as it is no longer needed. Also updated the package comment in parse.go to remove the nolint directive. * Add API package and update linter config Introduces a new api.go file for the SwanLab API, updates the .golangci.yml to configure revive linter for 'API' variable naming, and adds a nolint directive in parse.go to suppress revive warnings for the package name. * Remove unused comment and fix package declaration Cleaned up the package declaration by removing the nolint directive and an extraneous 'revive' line in parse.go. * Update revive linter rules in golangci config Removes the custom var-naming rule for 'API' in the revive linter and adds a new exclusion for var-naming warnings in the internal/api directory.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )