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

USAGOV-2197 phpstan ssg postprocessing #2201

Merged
merged 20 commits into from
Feb 3, 2025

Conversation

arpage
Copy link
Collaborator

@arpage arpage commented Jan 29, 2025

Jira Task

https://cm-jira.usa.gov/browse/USAGOV-2197

Description

Adding type hints and/or Drupal docblocks/tags to eliminate lint warnings

Type of Changes

  • New Feature
  • Bugfix
  • Frontend (Twig, Sass, JS)
    • Add screenshot showing what it should look like
  • Drupal Config (requires "drush cim")
  • New Modules (requires rebuild)
  • Documentation
  • Infrastructure
    • CMS
    • WAF
    • WWW
    • Egress
    • Tools
    • Cron
  • Other

Testing Instructions

Change Requirements

  • Requires New Documentation (Link: {})
  • Requires New Config
  • Requires New Content

Validation Steps

Ensure static site generation still occurs, and the drush usapubcsv command is still executed successfully (see tome logs).

Security Review

  • Adds/updates software (including a library or Drupal module)
  • Communication with external service
  • Changes permissions or workflow
  • Requires SSPP updates

Reviewer Reminders

  • Reviewed code changes
  • Reviewed functionality
  • Security review complete or not required

Post PR Approval Instructions

Follow these steps as soon as you merge the new changes.

  1. Go to the USAGov Circle CI project.
  2. Find the commit of this pull request.
  3. Build and deploy the changes.
  4. Update the Jira ticket by changing the ticket status to Review in Test and add a comment. State whether the change is already visible on cms-dev.usa.gov and beta-dev.usa.gov, or if the deployment is still in process.

@arpage arpage requested review from omerida and akf January 29, 2025 18:55
Copy link
Contributor

@omerida omerida left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple of places where typehints can be more specific and minimize use of mixed

@omerida omerida merged commit 8ba84c7 into dev Feb 3, 2025
4 of 5 checks passed
@omerida omerida deleted the USAGOV-2197-phpstan-ssg-postprocessing branch February 3, 2025 16: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.

2 participants