Skip to content

Conversation

drupol
Copy link
Owner

@drupol drupol commented Apr 29, 2025

No description provided.

@drupol drupol force-pushed the push-utnosqkwqsrl branch 18 times, most recently from e2da0b0 to 20bbfb7 Compare April 30, 2025 13:26
@mightyiam
Copy link

This is not a small number of files.

@drupol
Copy link
Owner Author

drupol commented Apr 30, 2025

The huge diff is mostly because of the facter.json files.
Now, the amount of files is bigger because I can split everything is smaller modules.
I still need to migrate stuff from the folder _to_migrate to the new structure... but I do that slowly as I need to generate the facter.json files on the machine, which I don't have access all the time.

@drupol drupol force-pushed the push-utnosqkwqsrl branch from 20bbfb7 to 9ffaf66 Compare April 30, 2025 15:16
@mightyiam
Copy link

Yes, I know how time consuming this can be. I hope you eventually feel it was worth it.

@drupol drupol force-pushed the push-utnosqkwqsrl branch 8 times, most recently from 9695f39 to 1c4441a Compare May 1, 2025 21:10
@drupol drupol force-pushed the push-utnosqkwqsrl branch 4 times, most recently from 7e96a61 to 4dd94db Compare May 4, 2025 21:06
@NotAShelf
Copy link

"minor refactor"

got here from this post, it was a fun read :)

@drupol drupol force-pushed the push-utnosqkwqsrl branch 19 times, most recently from c3104d9 to b96e30b Compare May 12, 2025 07:43
@drupol drupol force-pushed the push-utnosqkwqsrl branch from b96e30b to f4f0eeb Compare May 14, 2025 10:53
@drupol drupol merged commit 3523873 into master May 14, 2025
2 checks passed
@mightyiam
Copy link

Congrats 🎉

@drupol drupol deleted the push-utnosqkwqsrl branch May 14, 2025 11:54
@drupol
Copy link
Owner Author

drupol commented May 14, 2025

Thank you ! :)

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