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

Combine Folders Feature Added #115

Closed
wants to merge 8 commits into from

Conversation

danny007in
Copy link

Can combine folders with another folder and lint within as same array result

Normal without combine

fusv build/scss node_modules/bootstrap/scss

FdsKcLcIZc

With combine Feature

fusv build/scss node_modules/bootstrap/scss -c

UG46BlC9X7

cli.js Show resolved Hide resolved
@danny007in
Copy link
Author

can merge

@gentzeng
Copy link

I like this feature. I am currently trying to rebase it with PR #125 to avoid conflicts. Hope you don't mind

@danny007in
Copy link
Author

danny007in commented Jan 30, 2021

I like this feature. I am currently trying to rebase it with PR #125 to avoid conflicts. Hope you don't mind

No dont do now plz,
Because #125 as some conflicts

@gentzeng
Copy link

Year, that is exactly what I meant. I just fetched your PR #115 to my local repository to update it and to propose it to you. I'm not changing anything here and will notice you, when my proposal is ready for your review

@danny007in
Copy link
Author

danny007in commented Jan 30, 2021

Year, that is exactly what I meant. I just fetched your PR #115 to my local repository to update it and to propose it to you. I'm not changing anything here and will notice you, when my proposal is ready for your review

I am editing here #124 so wait

@gentzeng
Copy link

gentzeng commented Jan 30, 2021

Alright

@danny007in
Copy link
Author

does this feature implemented in main branch?

@XhmikosR
Copy link
Owner

I don't think it is. You could try rebasing and maybe @GeoSot or I could have a look later.

@XhmikosR
Copy link
Owner

This needs a proper rebase and make sure that lint and tests pass. Feel free to make a proper, new PR.

@XhmikosR XhmikosR closed this Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants