#620 Update .prettierrc #686

Closed
a24nedka wants to merge 1 commit from #620-update-prettier-config into team_2_week_4
Collaborator

The Prettier configuration has been updated to better align with the CSS conventions.
It ensures:

  • 2-space indentation
  • Double quotes
  • Proper spacing in declarations
  • Semicolons at the end of declarations

Made testfile to see if it works.

Note:
Some rules defined in the CSS guidelines (wiki) cannot be enforced automatically by Prettier and must be followed manually.

image
image

The Prettier configuration has been updated to better align with the CSS conventions. It ensures: - 2-space indentation - Double quotes - Proper spacing in declarations - Semicolons at the end of declarations Made testfile to see if it works. Note: Some rules defined in the CSS guidelines (wiki) cannot be enforced automatically by Prettier and must be followed manually. ![image](/attachments/227cd84f-521e-490b-a1de-8428dff59e24) ![image](/attachments/3ead31ae-11e5-4e98-9160-731e7dedb3fb)
a24nedka changed title from Update .prettierrc to #620Update .prettierrc 2026-04-30 16:46:04 +00:00
a24nedka changed title from #620Update .prettierrc to #620 Update .prettierrc 2026-04-30 16:46:19 +00:00
Collaborator

Review

Whats done

  • Checked to see that it works as intended.

What needs to be done

  • The configuration should only apply to CSS files. Right now, all files change according to the new configuration.

Conclusion

  • We could simply overwrite to have the indentation only applied to CSS, commented in issue #710.
# Review ## Whats done - Checked to see that it works as intended. ## What needs to be done - The configuration should only apply to CSS files. Right now, all files change according to the new configuration. ## Conclusion - We could simply overwrite to have the indentation only applied to CSS, commented in issue https://git.webug.se/Andras/BoundlessFlowCampus2K/issues/710.
Collaborator

The issue is not ready to be merge. Before merge change so that the indentation is only overwritten for css. Possible solution can be found here #710.

The issue is not ready to be merge. Before merge change so that the indentation is only overwritten for css. Possible solution can be found here #710.
Collaborator

As we have noticed inconsistencies in how prettier behaves we have decided to research it during next week and also wait with any merges involving prettier.

As we have noticed inconsistencies in how prettier behaves we have decided to research it during next week and also wait with any merges involving prettier.
c24danli 2026-05-06 06:32:58 +00:00

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
4 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Andras/BoundlessFlowCampus2K!686
No description provided.