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

[doc] Remove incompatible web service project #5554

Merged
merged 1 commit into from
Jul 16, 2022
Merged

[doc] Remove incompatible web service project #5554

merged 1 commit into from
Jul 16, 2022

Conversation

Rotzbua
Copy link
Contributor

@Rotzbua Rotzbua commented Jul 12, 2022

The mentioned project has problems and should be removed because:

  • Abandoned since 2017
  • Not compatible with Python3

@wRAR
Copy link
Member

wRAR commented Jul 12, 2022

You should also remove it from index.rst (not sure if there are other changes needed).

* Abandoned since 2017
* Not compatible with Python3
@Rotzbua
Copy link
Contributor Author

Rotzbua commented Jul 12, 2022

@wRAR Thanks for the hint. I found references just in the index.rst.

@wRAR wRAR added the docs label Jul 12, 2022
@codecov
Copy link

codecov bot commented Jul 12, 2022

Codecov Report

Merging #5554 (1c7ed4f) into master (4994202) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5554   +/-   ##
=======================================
  Coverage   88.73%   88.73%           
=======================================
  Files         162      162           
  Lines       10774    10774           
  Branches     1843     1843           
=======================================
  Hits         9560     9560           
  Misses        940      940           
  Partials      274      274           

@kmike
Copy link
Member

kmike commented Jul 16, 2022

Thanks @Rotzbua!

@kmike kmike merged commit 94a0324 into scrapy:master Jul 16, 2022
@Rotzbua Rotzbua deleted the doc-remove-outdated branch July 17, 2022 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants