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

WIP: Move codes indexer as a sub package #6917

Closed
wants to merge 8 commits into from

Conversation

lunny
Copy link
Member

@lunny lunny commented May 12, 2019

No description provided.

@lunny lunny added type/enhancement An improvement of existing functionality type/refactoring Existing code has been cleaned up. There should be no new functionality. labels May 12, 2019
@lunny lunny force-pushed the lunny/code_indexer_interface branch from 75a6d02 to 857446f Compare May 14, 2019 12:13
// Use of this source code is governed by a MIT-style
// license that can be found in the LICENSE file.

package codes
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Plural form of the word code is code. Could you change the name of this package to code?

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label May 15, 2019
@stale
Copy link

stale bot commented Jul 14, 2019

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 months. Thank you for your contributions.

@stale stale bot added the issue/stale label Jul 14, 2019
@lunny lunny added the issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented label Jul 14, 2019
@stale stale bot removed the issue/stale label Jul 14, 2019
@lunny
Copy link
Member Author

lunny commented Dec 20, 2019

replaced by #9313

@lunny lunny closed this Dec 20, 2019
@lunny lunny deleted the lunny/code_indexer_interface branch December 20, 2019 12:15
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. type/enhancement An improvement of existing functionality type/refactoring Existing code has been cleaned up. There should be no new functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants