Skip to content

fix: files.ignore setting should work #462

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

Merged
merged 13 commits into from
Jul 19, 2025
Merged

fix: files.ignore setting should work #462

merged 13 commits into from
Jul 19, 2025

Conversation

juleswritescode
Copy link
Collaborator

@juleswritescode juleswritescode commented Jul 18, 2025

So I took a look at the problem at hand.

Here it is:

image

I then went back to my garage and sifted through my tools – until I found it.



The perfect tool for the job.



My choice came down to this:

image



It does the job just right.



Okay, jokes aside, I just wanted to write a macro as an exercise – we can also just add a couple lines to every workspace function 😉

@juleswritescode juleswritescode changed the title Fix/439 fix: files.ignore setting should work Jul 18, 2025
@juleswritescode juleswritescode marked this pull request as ready for review July 18, 2025 09:03
Copy link
Collaborator

@psteinroe psteinroe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as the guy that spent month trying to get change.rs to work I have no right to not approve this monster.

@juleswritescode
Copy link
Collaborator Author

juleswritescode commented Jul 19, 2025

as the guy that spent month trying to get change.rs to work I have no right to not approve this monster.

when the macro becomes annoying to maintain, let's not feel bad and get rid of it quickly 😛

@juleswritescode juleswritescode enabled auto-merge (squash) July 19, 2025 07:27
@juleswritescode juleswritescode merged commit fa18370 into main Jul 19, 2025
7 checks passed
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.

2 participants