-
Notifications
You must be signed in to change notification settings - Fork 82
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
Minor fixes and whitespace #2896
Conversation
Will the NR changes revert if changed in BDData Editor? |
I need to test if the NR changes will revert. |
Battlescribe does still save numbers as 1.0 while NR saves as 1, etc. |
I'm way to used to the old editor. |
@Mayegelt they are basically the same, the NR one just has a bunch of QoL features that make developing easier. I'd strongly recommend checking it out. I was wary, but the transition was basically seamless. Plus, having an active dev means that any defects are fixed quickly, often within hours. |
That's what I did. |
An issue I could see with switching is that other programs that also use the BS data might end up being affected. So I would also ideally not wish to move away to one alternative that could inadvertently cause issues for others programs. |
I'll try to do a review of the push later this week if I can. Still not very well. But I'm seeing the issues piling up and most should be fixable in a couple of hours work. |
@nstephenh Also I saw you put a catalogue link in DA to Mech Library. This is not required as the file is already linked to the LA library, which links to the Mech Library. So now it is double linked. It doesn't cause any issues. But it is just unnecessary. It seems like a few of the pts changes you made from a number with a ".0" or whatever to no longer having that. There are a lot in Dark Angels & Mech and a couple in Salamanders. & World Eaters I'm not sure if you left them in deliberately, or if they will cause any disruption on BS or other programs. |
Remade this as #2905 |
Just a few minor fixes.
Not switching to newrecruit at this time, but I did run some of these files through NR which trimmed some excess whitespace.
Fixed Units
Secutors
Vulkan
Excindo battle automata
Red Hand Destroyer Ravager