FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

JCL-331: Convenience modules for integrators by acoburn · Pull Request #416 · inrupt/solid-client-java · GitHub

Repository navigation

JCL-331: Convenience modules for integrators - #416

Merged
acoburn merged 1 commit into
mainfrom
JCL-331-convenience-artifacts
Apr 11, 2023
Merged

acoburn merged 1 commit into
mainfrom
JCL-331-convenience-artifacts

Conversation

acoburn commented Apr 10, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor

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 requested a review from a team April 10, 2023 18:31
acoburn temporarily deployed to ESS PodSpaces April 10, 2023 18:31 — with GitHub Actions Inactive
acoburn temporarily deployed to ESS PodSpaces April 10, 2023 18:31 — with GitHub Actions Inactive
acoburn temporarily deployed to ESS PodSpaces April 10, 2023 18:31 — with GitHub Actions Inactive
acoburn temporarily deployed to ESS DevNext April 10, 2023 18:31 — with GitHub Actions Inactive

acoburn commented Apr 10, 2023

Copy link
Copy Markdown
Contributor Author

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

This branch was previously deployed

2 inactive deployments
ESS DevNext — e798bd32 Deployed Apr 11, 2023 by acoburn
ESS PodSpaces — e798bd32 Deployed Apr 11, 2023 by acoburn
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL