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.

Sign up for GitHub

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

Jump to bottom

Add encodeLib to build sequence, or delete it #642

Closed
xrchz opened this issue May 9, 2019 · 2 comments
Closed

Add encodeLib to build sequence, or delete it #642

xrchz opened this issue May 9, 2019 · 2 comments
Assignees
@mn200 @acjf3 @myreen

Comments

@xrchz
Copy link
Member

xrchz commented May 9, 2019

compiler/encoders/tests/ is never used anywhere and does not work. It should be turned into a test theory that can be run as part of the regression test and added to the build sequence. Or if that is not desired, it should be deleted.
@xrchz
Copy link
Member Author

xrchz commented Dec 9, 2024

@acjf3 how could this be tested?

@dnezam
Copy link
Contributor

dnezam commented Dec 14, 2024

Closing in favor of #1116 which aims to locate these files automatically and thus applies more generally.

@dnezam dnezam closed this as completed Dec 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mn200 mn200

@acjf3 acjf3

@myreen myreen

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@mn200 @xrchz @acjf3 @myreen @dnezam