David RACODON comments

Results 41 comments of


                                            David RACODON

🧪 [linty] initial setup (preliminary)

> Unfortunately not yet. I have already checked the permissions, but everything seems to be correct. Maybe we should just add an `ls -al` to the job to see if...

🧪 [linty] initial setup (preliminary)

You need to add `if: always()` to the step to make it run even if the previous step failed: ``` - name: Test if: always() run: ls -al ```

🧪 [linty] initial setup (preliminary)

> Anyway. Linty fails since we have changed the top entity. In the previous (working) top entity **all** generics were defined/set, while in the current one (`neorv32_top`) some are undefined/unset....

🧪 [linty] initial setup (preliminary)

> `run` and `uses` cannot be used within the same job step. Sure. You need a dedicated step for the `ls`. > I just tested to use a testbench as...

🧪 [linty] initial setup (preliminary)

@stnolting : I finally found out the issue about the `.linty` directory not being uploaded. Here's the fix: https://github.com/Linty-Services/bugfinder-sample/commit/e283ca4aca80447f95f14fbeed4762c67950b7f7

🧪 [linty] initial setup (preliminary)

> Thank you very much! > > I'm quite busy at the moment, but I'll come back to this ASAP (wekkend)! 🙈 Great!

🧪 [linty] initial setup (preliminary)

> Hmm, maybe this is just a code issue... > > ``` > [00000.140434] 58. Executing hlp_synthesized_objects pass... > [[VHDL-1769;./../rtl/core/neorv32_top.vhd;25;5;25;20]] VERIFIC-ERROR [VHDL-1769] ./../rtl/core/neorv32_top.vhd:25: formal generic 'clock_frequency' has no actual or...