Skip to content

chore: format code using Prettier#36

Merged
nicomiguelino merged 2 commits intoScreenly:masterfrom
nicomiguelino:fix-35
Mar 5, 2025
Merged

chore: format code using Prettier#36
nicomiguelino merged 2 commits intoScreenly:masterfrom
nicomiguelino:fix-35

Conversation

@nicomiguelino
Copy link
Copy Markdown
Collaborator

@nicomiguelino nicomiguelino commented Mar 5, 2025

Description

How Has This Been Tested?

  • Unit tests
  • Manual tests

Checklist

  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation (where applicable).
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@nicomiguelino nicomiguelino requested a review from Copilot March 5, 2025 20:40
Copy link
Copy Markdown

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.

PR Overview

This PR applies Prettier formatting across the repository to enforce consistent code style. The changes update the formatting in documentation, workflows, tests, and production code without impacting functionality.

  • Updated markdown and documentation files for consistent bullet points and formatting.
  • Reformatted JavaScript source code and test files with standardized commas, spacing, and line breaks.
  • Adjusted the GitHub Actions workflow file to align with expected YAML structure.

Reviewed Changes

File Description
CONTRIBUTING.md Adjusted spacing and code block formatting for branch instructions.
.github/workflows/check-workflow-syntax.yml Removed a stray dash to better match GitHub Actions YAML structure.
.github/pull_request_template.md Changed bullet style for issue references.
docs/cookbook/README.md Standardized link formatting for asset uploads.
test/complete-workflow.test.js Reformatted nock calls and response arrays for clarity.
index.js Broke a long helpText into two lines per Prettier standards.
actions/complete-workflow.js Reformatted request headers for consistency.
docs/common-use-cases.md Added spacing for improved readability.
docs/cookbook/uploading-an-asset-via-google-drive.md Updated bullet points and flow for trigger and action configuration.
docs/cookbook/uploading-an-asset-via-dropbox.md Similar formatting adjustments as the Google Drive document.
test/cleanup.test.js Reformatted nock responses, adding necessary commas.
actions/cleanup-zapier-content.js Minor arrow function formatting adjustments for readability.

Copilot reviewed 26 out of 26 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

.github/workflows/check-workflow-syntax.yml:1

  • The stray dash on the first line may cause YAML parsing errors. Please remove it to maintain valid workflow syntax.
 -

@nicomiguelino nicomiguelino requested a review from vpetersson March 5, 2025 20:47
@nicomiguelino nicomiguelino merged commit 7421abf into Screenly:master Mar 5, 2025
2 checks passed
@nicomiguelino nicomiguelino deleted the fix-35 branch March 5, 2025 23:44
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.

[CHORE] Format code using Prettier

3 participants