Skip to content
This repository has been archived by the owner on Sep 3, 2024. It is now read-only.

Regression tests using outdated version of Veryl. #18

Closed
nblei opened this issue Aug 31, 2024 · 2 comments
Closed

Regression tests using outdated version of Veryl. #18

nblei opened this issue Aug 31, 2024 · 2 comments

Comments

@nblei
Copy link
Contributor

nblei commented Aug 31, 2024

  × Unexpected token: ':'
   ╭─[/home/runner/work/std/std/src/async_fifo/async_fifo.veryl:7:26]
 6 │ /// Generic parameter `S` is a module name of synchrinozer.
 7 │ pub module async_fifo::<S: synchronizer> #(
   ·                          ┬
   ·                          ╰── Error location
 8 │     param WIDTH             : u32  = 8                         , /// Data width
   ╰────
@dalance
Copy link
Contributor

dalance commented Sep 2, 2024

Yes. This is one of motivations of #17.
I'll change Veryl of CI from stable release to github temporaly.
After merging #6 and #19, I'll do #17.

@dalance
Copy link
Contributor

dalance commented Sep 2, 2024

Regression failure was fixed.

@dalance dalance closed this as completed Sep 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants