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

[Docs/Website] DataTypes article mentions Interval types #3325

Closed
mwachs5 opened this issue Jun 6, 2023 · 0 comments · Fixed by #3345
Closed

[Docs/Website] DataTypes article mentions Interval types #3325

mwachs5 opened this issue Jun 6, 2023 · 0 comments · Fixed by #3345
Labels
good first issue An issue whose fix is simple. Perfect for a new developer wanting to get involved!

Comments

@mwachs5
Copy link
Contributor

mwachs5 commented Jun 6, 2023

Type of issue: Bug Report

Please provide the steps to reproduce the problem:

Look at the website about DataTypes: https://www.chisel-lang.org/chisel3/docs/explanations/data-types.html

What is the current behavior?

It says:
Screen Shot 2023-06-06 at 10 06 42 AM

And there is still the article about Interval Type under experimental features: https://www.chisel-lang.org/chisel3/docs/appendix/experimental-features#interval-type

What is the expected behavior?

There should be no mention of IntervalTypes because they are no longer supported.

Source for first issue is here: https://github.com/chipsalliance/chisel/blob/main/docs/src/explanations/data-types.md

Second issue is here: https://github.com/chipsalliance/chisel/blob/main/docs/src/appendix/experimental-features.md

Please tell us about your environment:

Other Information

What is the use case for changing the behavior?

Less user confusion

@mwachs5 mwachs5 added the good first issue An issue whose fix is simple. Perfect for a new developer wanting to get involved! label Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue An issue whose fix is simple. Perfect for a new developer wanting to get involved!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant