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

Feature/refactor all #2

Merged
merged 12 commits into from
Jun 25, 2021
Merged

Feature/refactor all #2

merged 12 commits into from
Jun 25, 2021

Conversation

geoheelias
Copy link
Contributor

I think even if there is more to clean out/refactor - right now is a currently matching crawler-indexer relationship, and it makes sense to merge.

GitLab crawler still needs to be refactored to work against the current indexer and state setup, same with bitbucket if want to keep that (low prio ofc.).

Another point is, do we want to keep all the unused github crawlers (and bitbucket)? They do contain good information as to what we can/cant do as a note of "research" of sorts. Im not sure? I guess we keep them and litter the repo-structure.

@geoheelias geoheelias requested a review from puhoy June 25, 2021 00:54
@puhoy puhoy merged commit a0c4f9e into master Jun 25, 2021
@puhoy puhoy deleted the feature/refactor_all branch June 25, 2021 10:46
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.

None yet

3 participants