Skip to content

Conversation

@Anushre2005
Copy link

This PR renames the comment system test file and updates the test class name
to match the filename and follow the project’s naming conventions.

Fixes #11620

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

  • PR is descriptively titled 📑 and links the original issue above 🔗
  • tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with rake test
  • code is in uniquely-named feature branch and has no merge conflicts 📁
  • screenshots/GIFs are attached 📎 in case of UI updation (not applicable)
  • ask @publiclab/reviewers for help, in a comment below

@Anushre2005
Copy link
Author

@publiclab/reviewers Hi 👋 This is my first open-source contribution.
I’ve completed the requested changes — please let me know if anything needs updating.
Thank you!

@Anushre2005
Copy link
Author

Hi 👋
The CI failures appear to be related to GitHub Actions / Codecov infrastructure
(deprecated actions and rate limiting), and not caused by this change.
Please let me know if you’d like me to update anything on my end. Thanks!

@Anushre2005
Copy link
Author

Hi maintainers 👋
It looks like the CI is failing due to the repository workflow using the deprecated
actions/upload-artifact@v2, which GitHub now blocks automatically.

The failures seem unrelated to the PR changes themselves.
Happy to submit a workflow-fix PR (upgrade to @v4) if that’s helpful.
Thanks!

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.

Solve class mismatch error

1 participant