Skip to content

6609 update class name#416

Closed
vasconsaurus wants to merge 52 commits intodevelopfrom
6609-update-class-name
Closed

6609 update class name#416
vasconsaurus wants to merge 52 commits intodevelopfrom
6609-update-class-name

Conversation

@vasconsaurus
Copy link
Contributor

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context – why has this been changed/fixed.

References: TICKET-ID, TICKET-ID, …

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can verify the changes. Please describe whether or not you implemented automated tests.

Things to pay attention to during code review

Please describe parts of the change that require extra attention during code review, for example:

  • File FFFF, line LL: This refactoring does this and this. Is it consistent with how it’s implemented elsewhere?
  • Etc.

Checklist

  • I have performed a self-review of my own code
  • I have added unit and feature tests, if the PR implements a new feature or otherwise would benefit from additional testing
  • I have added regression tests, if the PR fixes a bug
  • I have added logging, exception reporting, and custom tracing with any additional information required for debugging
  • I considered secure coding practices when writing this code. Any security concerns are noted above.
  • I have commented my code in hard-to-understand areas, if any
  • I have made needed changes to the README
  • My changes generate no new warnings

caiosba and others added 30 commits December 2, 2022 20:36
- We are deprecating multiple parsers. They are either using the Zapier RSS/Check integration or the API.
        - 20 minutes - RSS
        - Estadao - RSS
        - THIP - RSS
        - PesaCheck - API
        - Comprova - API
        - Newtral - API (both newtral_fact_check and newtral_fakes)
        - CekFakta - API (tempo_cekfakta parser)
        - Tempo - API (tempo_cekfakta parser)
- Updating the README, so the how to deprecate is more visible.

References: 4383, 4349, 4347, 4300
PR: #384
It's now using RSS/Check Zapier integration to import fact checks.

Note: We also had to update a test that was using the mafindo parser.

Ticket: 4401
PR: #386
Vishvas News fact-checks are now being imported through Zapier.
Newschecker is now importing fact-checks through the zapier/check integration.
It is now importing fact-checks through the zapier/check integration.

References: 4402
It is now importing fact-checks through the zapier/check integration.

References: 4460
vasconsaurus and others added 22 commits April 30, 2024 14:11
* It is now importing fact-checks through the zapier/check integration.

References: 4494
Boom parsers are now importing fact-checks through the zapier/check integration.

References: 4466
It is now importing fact-checks through the zapier/check integration.

References: 4525
PR: #395
Uol and Comprova are now using the API, so we can deprecate their parser.

References: 4403
PR: 396
Deprecates:
CV2-5046 – We are now importing via the Zapier/Check integration:
1. Piyaoba

CV2-4981 – We confirmed the parsers below can be deprecated:
1. aajtak_hindi – can be deprecated ([link](https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=50881 "https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=50881"))
2. africa_check – can be deprecated ([link](https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=50881 "https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=50881"))
3. agência lupa – can be deprecated ([link](https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=50850 "https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=50850"))
4. alt_news_in – can be deprecated ([link](https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=50881 "https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=50881"))
5. fact_check_cite – can be deprecated ([link](https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=50850 "https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=50850"))
6. globo – can be deprecated ([link](https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=50881 "https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=50881"))
7. la_silla_vacia – can be deprecated ([link](https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=50881 "https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=50881"))
8. vera_files – can be deprecated ([link](https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=51344 "https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=51344"))
9. vera_files_filipino – can be deprecated ([link](https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=51344 "https://meedan.atlassian.net/browse/CV2-4981?focusedCommentId=51344"))

References: 5046, 4981
PR: 401
1. climate_fact_check
2. gesis_claims
3. tfc_taiwan
Deprecate the Tsek.ph parser. We are now using another integration.

References: CV2-6221
Error: This request has been automatically failed because it uses a
deprecated version of `actions/cache: v2`. Please update your workflow
to use v3/v4 of actions/cache to avoid interruptions. Learn more:
https://github.blog/changelog/2024-12-05-notice-of-upcoming-releases-and-breaking-changes-for-github-actions/#actions-cache-v1-v2-and-actions-toolkit-cache-package-closing-down
It was confirmed by program that these two parsers are no longer used.

References: CV2-6236
Code Climate has been discontinued following its acquisition. We need to fully remove its integration from Fetch. This ticket aims to only do that.

Note: I also removed DEPLOY_ENV=travis, since we no longer use Travis.

References: CV2-6485, CV2-6593
Deprecate research parsers.

References: CV2-6565
Deprecate parsers.

References: CV2-6565

Sidenote: This test was failing: spec/lib/db_integration_test.rb. Now that we have only a couple partners, tests were running too fast and ClaimReview was returning nil, which was causing tests to fail. I did the easiest thing I could and added a sleep, with the least amount of time I could, which fixed this.
* It is now importing fact-checks through the zapier/check integration.

References: 4494
ClaimReviewParser.parsers already returns only enabled parsers. So when we try `!ClaimReviewParser.parsers[service].deprecated` with a deprecated service, it returns nil.

Which causes:
"\"error_message\":\"undefined method `deprecated' for nil:NilClass\",
\"error_class\":\"NoMethodError\",\"failed_at\":1761238248.0907707,
\"retry_count\":6,\"retried_at\":1761239681.7711115}"}"
@vasconsaurus vasconsaurus deleted the 6609-update-class-name branch October 24, 2025 18:10
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.

4 participants