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

comparisons with core files #199

Open
torvista opened this issue Jan 26, 2023 · 2 comments
Open

comparisons with core files #199

torvista opened this issue Jan 26, 2023 · 2 comments
Labels
question A question about the template

Comments

@torvista
Copy link

It is a thankless task to keep the template's files up to date with the core files, requiring careful comparison. But its important maintenance.
This becomes much harder when the template's file may have been a bit de-loused/formatted/updated so it becomes more up-to-date than the core file to which it is being compared.
While in theory all these things can be pushed to the core, whether they get accepted or not depends on various factors/moods/complaints, the default setting seems to be if not broken, don't fix. Sometimes, maybe, depends...

My conclusion is that template files should mirror the default-template core files, warts and all, where there is no real change, just to give the comparison tools something common to hang off in both files and so we can see the few real BS changes and the core updates more clearly
That means a file review should be done on the core file first, submitting those minor changes and waiting for yay/nay. Slow and random.
Meanwhile, when I go through a BS file to add my personal mods, I should change it to mirror default_template, even if going a bit backwards...

Thoughts??

@lat9
Copy link
Owner

lat9 commented Jan 26, 2023

I agree, but part of the reason for the reformatting/refactoring is that many of the template files required a fair amount of 'surgery' to get them to play nice with the Bootstrap framework.

I found it difficult to make those changes with the various insertions as well as the 'failure to align' properly. FWIW, I do try to keep a handle on the core-file template changes (although I missed the one for #198).

@lat9 lat9 added the question A question about the template label Jan 26, 2023
@torvista
Copy link
Author

Since these code-formatting-only changes introduce in BS files should in general only happen once with subsequent mods being core->BS or BS-only, perhaps volunteer minions could PR these minor changes back to core weekly to try and keep Default-RC_Classic-BS in sync where possible?
It's brain-dead work, but leaves lat9 more time to do something brain-alive.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question A question about the template
Projects
None yet
Development

No branches or pull requests

2 participants