VHDLFormatter icon indicating copy to clipboard operation
VHDLFormatter copied to clipboard

Some fixes and a new feature

Open fenkes-ibm opened this issue 4 years ago • 4 comments

Improve indentation around multiline assignments, comma fixup feature, plus some minor updates.

fenkes-ibm avatar Feb 22 '21 19:02 fenkes-ibm

Let me know if you'd like me to create issues for the indentation fixes.

fenkes-ibm avatar Feb 22 '21 19:02 fenkes-ibm

Thanks for this PR. I noticed that one unit test failed (indent assignment statement (with comment)). How do you handle this case?

CONSTANT Vcc : SIGNAL := --logic 1 constant
'1';
CONSTANT zero4 : bit_vector(0 TO 3) :=--test
('0', '0', '0', '0');

The expected behavior would be:

CONSTANT Vcc : SIGNAL := --logic 1 constant
    '1';
CONSTANT zero4 : bit_vector(0 TO 3) := --test
    ('0', '0', '0', '0');

g2384 avatar Feb 27 '21 18:02 g2384

Sorry for the long delay, busy with other work stuff right now :/ I had failed to run the jest testcases so I missed this. I found the bug but while doing so uncovered an inconsistency in my approach that I'd like to fix before pushing another set of changes. Coming up soon!

fenkes-ibm avatar Mar 04 '21 19:03 fenkes-ibm

I've added the jest into the github action. So if you merge master into your pr, github will check for you automatically.

g2384 avatar Mar 05 '21 13:03 g2384