Skip to content

Comments

Adding instructions for using Prettier to push changes to the production site#32

Merged
jrgriffiniii merged 1 commit intomainfrom
updating-readme-prettier
Aug 25, 2025
Merged

Adding instructions for using Prettier to push changes to the production site#32
jrgriffiniii merged 1 commit intomainfrom
updating-readme-prettier

Conversation

@jrgriffiniii
Copy link
Contributor

No description provided.

…ion site; Disabling Prettier formatting checks for deployment Actions
@jrgriffiniii jrgriffiniii force-pushed the updating-readme-prettier branch from 1e0d31c to 8c71ed0 Compare August 25, 2025 19:01
@jrgriffiniii jrgriffiniii requested a review from Copilot August 25, 2025 19:02
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes automated Prettier formatting checks from the CI pipeline and adds manual instructions for developers to run Prettier before pushing changes to ensure proper code formatting.

  • Removes the "prettier" job from the GitHub Actions workflow matrix
  • Eliminates the automated Prettier formatting check step from CI
  • Adds documentation instructing developers to manually run Prettier before pushing commits

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
src/app/contact/page.tsx Applies Prettier formatting to consolidate multi-line description into single line
README.md Adds IMPORTANT section with manual Prettier instructions for developers
.github/workflows/nextjs.yml Removes automated Prettier checks from CI workflow

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

strategy:
matrix:
job: ["prettier", "build"]
job: ["build"]
Copy link

Copilot AI Aug 25, 2025

Choose a reason for hiding this comment

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

Removing automated Prettier checks from CI reduces code quality enforcement. Consider keeping the Prettier check but making it non-blocking (using continue-on-error: true) or adding it as a separate non-required status check to maintain consistent formatting across contributions.

Copilot uses AI. Check for mistakes.

### IMPORTANT

_Please ensure that, before pushing any commits which introduce changes to the site, that you run the following:_
Copy link

Copilot AI Aug 25, 2025

Choose a reason for hiding this comment

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

Remove the redundant 'that' in this sentence. It should read: 'Please ensure that, before pushing any commits which introduce changes to the site, you run the following:'

Suggested change
_Please ensure that, before pushing any commits which introduce changes to the site, that you run the following:_
_Please ensure that, before pushing any commits which introduce changes to the site, you run the following:_

Copilot uses AI. Check for mistakes.
@jrgriffiniii jrgriffiniii merged commit 79eafb9 into main Aug 25, 2025
1 check 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.

1 participant