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

Cypress slack notifications with other changes #763

Merged
merged 3 commits into from
Dec 13, 2024

Conversation

FahadDarw
Copy link
Contributor

Summary

This PR updates the package.json file in the Dfe.Academies.External.Web/CypressTests folder, making adjustments to the dependencies and scripts for improved functionality and consistency.

Changes Made

  1. Scripts Updates:

    • Consolidated and streamlined the generate:html:report script to combine and generate reports in a single command.
    • Removed duplicate entries for lint and generate:html:report scripts.
  2. Dependencies Adjustments:

    • Added typescript with version ^5.1.6 to the dependencies section.
    • Removed duplicate entries for cypress-multi-reporters and zaproxy in the devDependencies section.

Impact

  • Ensures a cleaner package.json file with no duplicate or redundant entries.
  • Improves maintainability by aligning dependencies and scripts.
  • Enhances clarity for future contributors and developers.

Testing

  • Verified that all Cypress tests run successfully with the updated configuration.
  • Confirmed that reports are generated as expected using the updated generate:html:report script.

@FahadDarw FahadDarw merged commit f221aab into master Dec 13, 2024
3 checks passed
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.

2 participants