Skip to content

fix/remove workspace star#244

Open
abhishekjhagroww wants to merge 6 commits into
developfrom
fix/remove-workspace-star
Open

fix/remove workspace star#244
abhishekjhagroww wants to merge 6 commits into
developfrom
fix/remove-workspace-star

Conversation

@abhishekjhagroww

@abhishekjhagroww abhishekjhagroww mannequin commented Apr 28, 2023

Copy link
Copy Markdown
Mannequin

What does this PR do?

  • removed workspace:* from all packages and replaced with absolute versions

What packages have been affected by this PR?

ui-toolkit
monitoring
eslint-config
web-storage
analytics

Types of changes

What types of changes does your code introduce to this project?

Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)

  • New feature (non-breaking change which adds functionality)

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Package version increase in any of the packages?

ui-toolkit -> 0.3.2

Checklist before merging

Put an x in the boxes that apply

  • These changes have been thoroughly tested.

  • Changes need to be immediately published on npm.

},
"homepage": "https://github.com/Groww/webster/tree/develop/packages/monitoring#readme",
"devDependencies": {
"@groww-tech/eslint-config": "workspace:*",

Copy link
Copy Markdown
Mannequin Author

Choose a reason for hiding this comment

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

eslint has been moved to root, so not required here.

@samyak0472

samyak0472 mannequin commented May 5, 2023

Copy link
Copy Markdown
Mannequin

Why we are doing this? Any problem we faced?

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.

4 participants