-
Notifications
You must be signed in to change notification settings - Fork 2
fix: remove backup, add prettier #333
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
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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 backup files and adds Prettier for code formatting. The changes standardize code formatting across the project by introducing Prettier configuration and tooling while cleaning up obsolete backup directories.
- Removes extensive backup directory containing obsolete doPlugins, whoami, superbounce, and other legacy code files
- Adds Prettier as a development dependency with formatting scripts
- Includes Prettier VS Code extension recommendation for consistent editor experience
- Applies Prettier formatting to one active JavaScript file (braze_checkout_tracking.js)
Reviewed changes
Copilot reviewed 81 out of 85 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| package.json | Adds Prettier dependency and npm scripts for formatting |
| extensions/welt/braze_checkout_tracking.js | Reformatted with Prettier for improved readability |
| backup/* (multiple files) | Removed obsolete backup files |
| .vscode/extensions.json | Added Prettier VS Code extension recommendation |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 92 out of 96 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| 'anonymous': 'never', | ||
| 'named': 'never' | ||
| }], | ||
| 'space-before-function-paren': 'off', // Disabled to avoid conflict with Prettier |
Copilot
AI
Dec 16, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While the comment explains disabling this rule for Prettier, consider documenting which Prettier configuration is being used and ensuring the .prettierrc or equivalent configuration file is present in the repository to maintain consistency.
What does this change?
Removes extensive backup
adds prettier
adds extensions (moved from tealium)
Why?
Link to supporting ticket or Screenshots (if applicable)
https://axelspringer.atlassian.net/browse/TIPPI-1094