Skip to content

Conversation

@acoburn
Copy link
Collaborator

@acoburn acoburn commented Apr 10, 2023

While a modular codebase is very useful and flexible, it also makes it harder for someone who is new to the system to know where to begin.

This PR introduces a new artifact to make this easier:

  • inrupt-client-runtime -- an application developer can include this dependency in an application, and all of the recommended runtime libraries will be included.

* inrupt-client-minimal -- a library developer can include this dependency in their code. It is just like inrupt-client-all except it does not include the runtime modules.

@acoburn acoburn requested a review from a team as a code owner April 10, 2023 18:31
@acoburn acoburn temporarily deployed to ESS PodSpaces April 10, 2023 18:31 — with GitHub Actions Inactive
@acoburn acoburn temporarily deployed to ESS PodSpaces April 10, 2023 18:31 — with GitHub Actions Inactive
@acoburn acoburn temporarily deployed to ESS PodSpaces April 10, 2023 18:31 — with GitHub Actions Inactive
@acoburn acoburn temporarily deployed to ESS DevNext April 10, 2023 18:31 — with GitHub Actions Inactive
@acoburn
Copy link
Collaborator Author

acoburn commented Apr 10, 2023

Some additional conversation about this suggests dropping the inrupt-client-minimal artifact and changing *-all to one of:

  • inrupt-client-default
  • inrupt-client-runtime
  • inrupt-client-libs

timea-solid
timea-solid approved these changes Apr 11, 2023
@timea-solid timea-solid temporarily deployed to ESS PodSpaces April 11, 2023 08:55 — with GitHub Actions Inactive
@timea-solid timea-solid temporarily deployed to ESS PodSpaces April 11, 2023 08:55 — with GitHub Actions Inactive
@timea-solid timea-solid temporarily deployed to ESS DevNext April 11, 2023 08:55 — with GitHub Actions Inactive
@acoburn acoburn force-pushed the JCL-331-convenience-artifacts branch from 77c0268 to e798bd3 Compare April 11, 2023 13:50
@acoburn acoburn temporarily deployed to ESS PodSpaces April 11, 2023 13:51 — with GitHub Actions Inactive
@acoburn acoburn temporarily deployed to ESS PodSpaces April 11, 2023 13:51 — with GitHub Actions Inactive
@acoburn acoburn temporarily deployed to ESS DevNext April 11, 2023 13:51 — with GitHub Actions Inactive
@acoburn acoburn enabled auto-merge (squash) April 11, 2023 13:51
@acoburn acoburn merged commit 7a6e161 into main Apr 11, 2023
@acoburn acoburn deleted the JCL-331-convenience-artifacts branch April 11, 2023 14:10
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.

3 participants