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

Theia 7.44.0 build failed to compile #21223

Closed
nickboldt opened this issue Feb 28, 2022 · 7 comments
Closed

Theia 7.44.0 build failed to compile #21223

nickboldt opened this issue Feb 28, 2022 · 7 comments
Assignees
Labels
kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.

Comments

@nickboldt
Copy link
Contributor

nickboldt commented Feb 28, 2022

Describe the bug

Error: [compile] src/github-auth-plugin.ts(19,55): error TS2345: Argument of type '{ id: string; label: string; supportsMultipleAccounts: false; onDidChangeSessions: theia.Event<theia.AuthenticationProviderAuthenticationSessionsChangeEvent>; getSessions: () => Promise<...>; login: (scopes: string[]) => Promise<...>; logout: (id: string) => Promise<...>; }' is not assignable to parameter of type 'AuthenticationProvider'.
[compile]   Type '{ id: string; label: string; supportsMultipleAccounts: false; onDidChangeSessions: Event<AuthenticationProviderAuthenticationSessionsChangeEvent>; getSessions: () => Promise<...>; login: (scopes: string[]) => Promise<...>; logout: (id: string) => Promise<...>; }' is missing the following properties from type 'AuthenticationProvider': createSession, removeSession
[compile] error Command failed with exit code 2.

-- https://github.com/eclipse-che/che-theia/runs/5362752467?check_suite_focus=true

Che version

7.44.0

Steps to reproduce

  1. https://github.com/eclipse-che/che-theia/actions/workflows/release.yml
  2. GHA fails
  3. see log

Expected behavior

  1. https://github.com/eclipse-che/che-theia/actions/workflows/release.yml
  2. GHA passes
  3. branch created; new images pushed to Quay, new nodejs package published

Eclipse Che Logs

See https://github.com/eclipse-che/che-theia/actions/runs/1911409438

@nickboldt nickboldt added kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system. labels Feb 28, 2022
@nickboldt nickboldt assigned nickboldt and benoitf and unassigned nickboldt Feb 28, 2022
@nickboldt
Copy link
Contributor Author

nickboldt commented Feb 28, 2022

Failure could be related to eclipse-che/che-theia@7124cd9#annotation_2860286957 cc: @RomanNikitenko

@benoitf
Copy link
Contributor

benoitf commented Feb 28, 2022

It's related to https://github.com/eclipse-che/che-theia/blob/main/plugins/github-auth-plugin/package.json#L20

might be safer to use 1.18.0-next.120 instead

@nickboldt
Copy link
Contributor Author

Fix pushed to 7.44.x branch and build triggered in https://github.com/eclipse-che/che-theia/actions/runs/1911874689 while we wait for the above PR to get checked and merged

@nickboldt
Copy link
Contributor Author

nickboldt commented Feb 28, 2022

aaaaand it failed again.

Error: [compile] src/github-auth-plugin.ts(19,55): error TS2769: No overload matches this call.
[compile]   Overload 1 of 3, '(provider: AuthenticationProvider): Disposable', gave the following error.
[compile]     Argument of type '{ id: string; label: string; supportsMultipleAccounts: false; onDidChangeSessions: theia.Event<theia.AuthenticationProviderAuthenticationSessionsChangeEvent>; getSessions: () => Promise<...>; login: (scopes: string[]) => Promise<...>; logout: (id: string) => Promise<...>; }' is not assignable to parameter of type 'AuthenticationProvider'.
[compile]       Type '{ id: string; label: string; supportsMultipleAccounts: false; onDidChangeSessions: Event<AuthenticationProviderAuthenticationSessionsChangeEvent>; getSessions: () => Promise<...>; login: (scopes: string[]) => Promise<...>; logout: (id: string) => Promise<...>; }' is missing the following properties from type 'AuthenticationProvider': createSession, removeSession
[compile]   Overload 2 of 3, '(provider: AuthenticationProvider): Disposable', gave the following error.
[compile]     Argument of type '{ id: string; label: string; supportsMultipleAccounts: false; onDidChangeSessions: theia.Event<theia.AuthenticationProviderAuthenticationSessionsChangeEvent>; getSessions: () => Promise<...>; login: (scopes: string[]) => Promise<...>; logout: (id: string) => Promise<...>; }' is not assignable to parameter of type 'AuthenticationProvider'.
[compile]       Type '{ id: string; label: string; supportsMultipleAccounts: false; onDidChangeSessions: Event<AuthenticationProviderAuthenticationSessionsChangeEvent>; getSessions: () => Promise<...>; login: (scopes: string[]) => Promise<...>; logout: (id: string) => Promise<...>; }' is missing the following properties from type 'AuthenticationProvider': createSession, removeSession
[compile] error Command failed with exit code 2.
[compile] info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
[compile] yarn compile exited with code 2

-- https://github.com/eclipse-che/che-theia/runs/5364053326?check_suite_focus=true

benoitf added a commit to eclipse-che/che-theia that referenced this issue Mar 1, 2022
Fix eclipse-che/che#21223

Change-Id: I0aba83da752936e53fca852aaea89d818df32817
Signed-off-by: Florent Benoit <fbenoit@redhat.com>
@benoitf
Copy link
Contributor

benoitf commented Mar 1, 2022

2 PRs: eclipse-che/che-theia#1326 and eclipse-che/che-theia#1327 (one for main branch and the other one for 7.44.x)

@nickboldt
Copy link
Contributor Author

@nickboldt
Copy link
Contributor Author

nickboldt commented Mar 1, 2022

both builds are green! thanks for the assist, @benoitf !

https://mattermost.eclipse.org/eclipse/pl/c4b891ht1pf5zcsgq3np4hnf4o

jonnyczi-codegiant pushed a commit to codegiant-developer/che-theia that referenced this issue Mar 25, 2022
eclipse-che#1326)

Fix eclipse-che/che#21223

Change-Id: I0aba83da752936e53fca852aaea89d818df32817
Signed-off-by: Florent Benoit <fbenoit@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

3 participants