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

[stable22] Fix scss deprecated syntax #1993

Merged
merged 1 commit into from
Dec 7, 2021
Merged

Conversation

julien-nc
Copy link
Member

This makes the compilation fail in some contexts.

@julien-nc julien-nc added bug Something isn't working 3. to review labels Dec 6, 2021
@julien-nc julien-nc added this to the Nextcloud 22.2.5 milestone Dec 6, 2021
@julien-nc
Copy link
Member Author

/compile amend /

@juliusknorr
Copy link
Member

Pushed a commit as the package.json allowed using node 16 and then there is no prebuilt libsass available for that node version and the compile command fails with that. Remaining question is why the local build is different from the one in the Node CI / build action, but we can try to compare when the compile action commited the expected result. Let's see

@juliusknorr
Copy link
Member

/compile

@juliusknorr
Copy link
Member

At least CI is stable with that but the local build is still different even with the exact same node/npm versions. Not sure why though, the diff is rather odd with just some mapping hash changes it seems, using git diff --text --word-diff=porcelain --word-diff-regex=.

@julien-nc
Copy link
Member Author

/compile amend

Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
@julien-nc julien-nc merged commit 2be7cf0 into stable22 Dec 7, 2021
@delete-merged-branch delete-merged-branch bot deleted the fix/compiling/stable22 branch December 7, 2021 14:23
@skjnldsv skjnldsv mentioned this pull request Jan 7, 2022
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants