Skip to content

[5.3] Silence the SASS deprecations#44870

Closed
dgrammatiko wants to merge 11 commits intojoomla:5.3-devfrom
dgrammatiko:5.3-dev/sass
Closed

[5.3] Silence the SASS deprecations#44870
dgrammatiko wants to merge 11 commits intojoomla:5.3-devfrom
dgrammatiko:5.3-dev/sass

Conversation

@dgrammatiko
Copy link
Contributor

Pull Request for Issue # .

Summary of Changes

Testing Instructions

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@laoneo
Copy link
Member

laoneo commented Feb 12, 2025

Can we silence them only for the vendor folder?

@dgrammatiko
Copy link
Contributor Author

Can we silence them only for the vendor folder?

Probably, let me check

@joomla-cms-bot joomla-cms-bot added the NPM Resource Changed This Pull Request can't be tested by Patchtester label Feb 12, 2025
@laoneo
Copy link
Member

laoneo commented Feb 12, 2025

The vue fixes are handled here #44852.

@dgrammatiko
Copy link
Contributor Author

The vue fixes are handled here

I didnt see that PR, will revert

@joomla-cms-bot joomla-cms-bot removed the NPM Resource Changed This Pull Request can't be tested by Patchtester label Feb 12, 2025
@laoneo
Copy link
Member

laoneo commented Feb 17, 2025

Would that mean that when we add a deprecation in one of these files that they are not visible in the build output?

@dgrammatiko
Copy link
Contributor Author

The scss files are still visible in the output of the tool, it's just for them we suppress the deprecations logs
Screenshot 2025-02-17 at 11 23 22

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.

7 participants