Skip to content
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

Bump i18next from 21.10.0 to 22.0.4 in /services/idp #4941

Merged
merged 3 commits into from Mar 21, 2023

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Oct 31, 2022

Bumps i18next from 21.10.0 to 22.0.4.

Release notes

Sourced from i18next's releases.

v22.0.4

  • Add type for Formatter.addCached 1862

v22.0.3

  • Make getFixedT type-safe 1860
  • types: Omit object from t function 1858

v22.0.2

  • [TypeScript] Add default value for TDefaultResult 1853

v22.0.1

  • Fix normalize type options 1850

v22.0.0

  • Make the translation function fully type-safe 1775

BREAKING: Since this is a major rewrite for TypeScript usage we decided to create a major version. For JavaScript users v22.0.0 is equivalent to 21.10.0

Changelog

Sourced from i18next's changelog.

22.0.4

  • Add type for Formatter.addCached 1862

22.0.3

  • Make getFixedT type-safe 1860
  • types: Omit object from t function 1858

22.0.2

  • [TypeScript] Add default value for TDefaultResult 1853

22.0.1

  • Fix normalize type options 1850

22.0.0

  • Make the translation function fully type-safe 1775

BREAKING: Since this is a major rewrite for TypeScript usage we decided to create a major version. For JavaScript users v22.0.0 is equivalent to 21.10.0

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@update-docs
Copy link

update-docs bot commented Oct 31, 2022

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@sonarcloud
Copy link

sonarcloud bot commented Nov 3, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@fschade
Copy link
Contributor

fschade commented Nov 3, 2022

need to test the compatibility, draft 4 now

@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Nov 14, 2022

A newer version of i18next exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged.

@settings settings bot removed the javascript label Nov 30, 2022
@mmattel
Copy link
Contributor

mmattel commented Mar 1, 2023

@fschade ping maybe you can re-review

@mmattel mmattel requested a review from fschade March 1, 2023 14:28
@micbar
Copy link
Contributor

micbar commented Mar 9, 2023

@dependabot rebase

@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Mar 9, 2023

Looks like this PR has been edited by someone other than Dependabot. That means Dependabot can't rebase it - sorry!

If you're happy for Dependabot to recreate it from scratch, overwriting any edits, you can request @dependabot recreate.

@mmattel
Copy link
Contributor

mmattel commented Mar 9, 2023

Solved by applying master (recent changes in react-redux by @kulmann):

<<<<<<< dependabot/npm_and_yarn/services/idp/i18next-22.0.4
  react-i18next: 11.17.4_lbe2taj3rsyaxmdd4qnhuc5z6q
  react-redux: 7.2.8_sfoxds7t5ydpegc3knd667wn6m
=======
  react-i18next: 11.17.4_iunkfnprfsbsde3j2gf563wwwi
  react-redux: 8.0.5_shvqehxrdrredipssav7euweq4
>>>>>>> master

@mmattel mmattel force-pushed the dependabot/npm_and_yarn/services/idp/i18next-22.0.4 branch from 44ba7a8 to a470f91 Compare March 9, 2023 11:35
@ownclouders
Copy link
Contributor

ownclouders commented Mar 9, 2023

💥 Acceptance test Web-Tests-ocis-smoke-ocis-storage-1 failed. Further test are cancelled...

dependabot bot and others added 2 commits March 9, 2023 16:54
Bumps [i18next](https://github.com/i18next/i18next) from 21.10.0 to 22.0.4.
- [Release notes](https://github.com/i18next/i18next/releases)
- [Changelog](https://github.com/i18next/i18next/blob/master/CHANGELOG.md)
- [Commits](i18next/i18next@v21.10.0...v22.0.4)

---
updated-dependencies:
- dependency-name: i18next
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@mmattel mmattel force-pushed the dependabot/npm_and_yarn/services/idp/i18next-22.0.4 branch from cb69cc5 to f93b0ff Compare March 9, 2023 15:54
@mmattel mmattel force-pushed the dependabot/npm_and_yarn/services/idp/i18next-22.0.4 branch from f93b0ff to 9770930 Compare March 9, 2023 16:19
@sonarcloud
Copy link

sonarcloud bot commented Mar 9, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@mmattel
Copy link
Contributor

mmattel commented Mar 9, 2023

@fschade @micbar @kulmann I made that the CI is passing now by solving the conflict. Could you take a look and check if it is mergeable?

@mmattel mmattel requested review from kulmann and micbar March 10, 2023 07:38
@fschade fschade marked this pull request as ready for review March 16, 2023 13:30
@mmattel
Copy link
Contributor

mmattel commented Mar 16, 2023

@kulmann As this is a major release, I would like to bring this to your attention.

According the source page (i18next/i18next/pull/1775), the breaking change is:
We'll no longer support typescript below v4.1

As I do not know if we are above v4.1 or not, I would like to handover the decision for pressing the merge button to you (or one from the web team) 😅

@JammingBen
Copy link
Contributor

According the source page (i18next/i18next/pull/1775), the breaking change is: We'll no longer support typescript below v4.1

As I do not know if we are above v4.1 or not, I would like to handover the decision for pressing the merge button to you (or one from the web team) 😅

Web itself is on a newer TypeScript version, however, the IDP seems to use 4.1.2 (see https://github.com/owncloud/ocis/blob/master/services/idp/package.json#L153). I can't say if or how that affects the login page, so I'd prefer to wait for feedback from @kulmann on this.

@JammingBen
Copy link
Contributor

In general we could align the TypeScript version with the Web TypeScript version. Although not necessary, it would be nice for consistency, no?

@kulmann
Copy link
Member

kulmann commented Mar 21, 2023

In general we could align the TypeScript version with the Web TypeScript version. Although not necessary, it would be nice for consistency, no?

Basically nice for language feature consistency, yes, but I don't see how this can be in an issue as the idp code is completely independent from web.

@kulmann kulmann merged commit e7fd4f9 into master Mar 21, 2023
@delete-merged-branch delete-merged-branch bot deleted the dependabot/npm_and_yarn/services/idp/i18next-22.0.4 branch March 21, 2023 09:04
ownclouders pushed a commit that referenced this pull request Mar 21, 2023
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Florian Schade <f.schade@icloud.com>
Co-authored-by: mmattel <github@diemattels.at>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants