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

Toolchain requirements #1975

Merged
merged 9 commits into from
Mar 13, 2024
Merged

Toolchain requirements #1975

merged 9 commits into from
Mar 13, 2024

Conversation

egekorkan
Copy link
Contributor

Before starting the work on better tooling, I want to document what we currently generate and should keep on generating. Everything with a black output arrow in https://github.com/w3c/wot-thing-description/blob/main/toolchain/wot-toolchain-bpmn.png is also here but there is more.

This is not fixing #1958.

@egekorkan egekorkan marked this pull request as ready for review March 6, 2024 15:47
@egekorkan
Copy link
Contributor Author

@relu91 please have a final look. We want to agree on this set of requirements next week. The concrete change to another toolchain will come afterwards.

@relu91
Copy link
Member

relu91 commented Mar 7, 2024

Looks fine, maybe an additional (optional) requirement could be "lightweightness". Lightweight tools are easier to install and usually are easier to understand. But of course, we should not compromise functionality and therefore might be just a nice to have or a criteria that help use to choose between two solutions.

@relu91
Copy link
Member

relu91 commented Mar 7, 2024

BTW reading the requirements I remember the validation folder. Do we want to tackle that in this discussion or is rather something orthogonal?

@egekorkan
Copy link
Contributor Author

In the TD Call:

@egekorkan egekorkan merged commit 1f615df into main Mar 13, 2024
1 check passed
@egekorkan egekorkan deleted the egekorkan-patch-3 branch March 13, 2024 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants