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

doxygen: riot.css modified by 'make doc' #8122

Closed
kaspar030 opened this issue Nov 23, 2017 · 8 comments · Fixed by #19665
Closed

doxygen: riot.css modified by 'make doc' #8122

kaspar030 opened this issue Nov 23, 2017 · 8 comments · Fixed by #19665
Labels
Area: build system Area: Build system Area: doc Area: Documentation Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Comments

@kaspar030
Copy link
Contributor

Calling make doc modifies riot.css in-place, making it show up in git's changed files list.
That's a little annoying. Can that be avoided?

@kaspar030 kaspar030 added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Area: doc Area: Documentation Area: build system Area: Build system labels Nov 23, 2017
@aabadie
Copy link
Contributor

aabadie commented Nov 23, 2017

Can that be avoided?

I had the same problem and I fixed it by uninstalling lessc from npm and install python-lesscpy via apt-get instead.

@miri64
Copy link
Member

miri64 commented Nov 23, 2017

Can that be avoided?

An alternative would be to switch over from Bootstrap 3 to Bootstrap 4, which uses SASS instead of LESS (which does the compilation live in-browser). Would need some time however.

@jcarrano
Copy link
Contributor

See #10249 for a fix to this issue.

@aabadie
Copy link
Contributor

aabadie commented Jul 16, 2020

Is this still an issue ?

@miri64
Copy link
Member

miri64 commented Jul 16, 2020

We did not change how the riot.css is generated, so I'd say, yes. Unless people don't see this as an issue anymore :-D.

@aabadie
Copy link
Contributor

aabadie commented Jul 16, 2020

I also had this problem in the past but now it doesn't show up anymore. That's why I'm asking.

@miri64
Copy link
Member

miri64 commented Jul 16, 2020

Do you still have LESS compiler installed? Does it incidentally generate the same CSS as the one at hand?

@aabadie
Copy link
Contributor

aabadie commented Jul 16, 2020

Do you still have LESS compiler installed?

Ah, that might be that: I don't have LESS compiler :)

@MrKevinWeiss MrKevinWeiss added this to the Release 2021.07 milestone Jun 21, 2021
@MrKevinWeiss MrKevinWeiss removed this from the Release 2021.07 milestone Jul 15, 2021
@bors bors bot closed this as completed in cfee27c May 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: build system Area: Build system Area: doc Area: Documentation Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
5 participants