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
Update DCR Dependencies (workspaces prep) #3443
Conversation
Size Change: +1.59 kB (0%) Total Size: 943 kB
ℹ️ View Unchanged
|
@OllysCoding we have published a new version of node-fetch ( |
@LinusU Cheers mate! |
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.
I just have some questions about removal of props - was this to do with the unused props error that you mentioned in your notes? They seem to be in use.
@@ -86,7 +86,6 @@ const Headline: React.FC<{ | |||
headlineText: string; | |||
standfirst: string; | |||
pillar: Theme; | |||
starRating?: number; |
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.
What's the reason that star rating has been removed as part of this work?
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.
This prop was not used anywhere within the topMetaLiveblog
component, after updating, ESLint identified it and threw an error as unused prop.
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.
(Despite it being passed in by a parent component, it isn't used within the component, making it redundant)
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.
Ah, got it.
@@ -97,7 +97,6 @@ export const ShowcaseInterview = () => ( | |||
theme: Pillar.Culture, | |||
}} | |||
tags={[]} | |||
isShowcase={true} |
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.
Just wondering why this is getting removed?
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 data is passed into the component, the prop isn't actually consumed anywhere within it, meaning it's essentially redundant, causing ESLint to throw an error
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.
👍🏻
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.
I noticed the following packages may need another update as they're not on their maximum semver version. Could you please have a look at them?
eslint-plugin-react
aws-sdk
@guardian/src-text-input
@guardian/src-text-area
@guardian/src-radio
@guardian/src-link
@guardian/src-icons
@guardian/src-foundations
@guardian/src-ed-lines
@guardian/src-button
…core due to unclear updates
Thanks @marjisound, for |
What does this change?
Update all DCR dependencies & dev dependencies in line with the allowances of the package.json.
Fixes related to package updates:
@ts-expect-error
for new typescript errors which couldn't be resolvedWhy?
When we pull DCR & AR into yarn workspaces in the root of the monorepo, our lock files will be lost. By upgrading our packages to the most recent minor/patch versions available, when a new lock file is created, we'll have already fixed any associated bugs.