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

From frontend-meet: Changes to the mono-repo #20

Open
15 of 29 tasks
AskeLange opened this issue Oct 10, 2022 · 0 comments
Open
15 of 29 tasks

From frontend-meet: Changes to the mono-repo #20

AskeLange opened this issue Oct 10, 2022 · 0 comments
Assignees

Comments

@AskeLange
Copy link
Contributor

AskeLange commented Oct 10, 2022

Present changes to the mono-repo, and related.

Todo:

  • Remove nuxt-link-ext from wip-packages.
  • Remove the screens package pull-request and issue.
  • Write about JSDocs in documentation and link to the docs.
  • Correct naming, especially in JS-packages, but do also use dev instead of serve.
  • Add package-issues for CookieInformationBlocker and UmbracoSvg.
  • See if we can dual-publish to yarn.

Updated Lerna

Updated Lerna to version 5.5.0 from version 4.0.0.

The only relevant breaking changes were that node v12 and previous were no longer supported. Fortunately we were already using lts/fermium (v14), so this shouldn't provide any issues. Our continuous integration workflow were using node v12 however, so this have been updated to reflect the version in the .nvmrc file. I've only been able to do surface-level testing without republishing a package, so if you run into any issues, feel free to poke at me.

Changes to Husky

Added yarn test to pre-commit husky check, just to make sure nothing is broken when committing.

Added issues for packages

See GitHub project: here.
Also added issue templates to the limbo-frontend repository.

Added the following:

Components

  • Accordion.
  • AutoComplete (missing description).
  • Dropdown.
  • LimboSearch.
  • LongRead (missing description).
  • SkipToContent.
  • SwiperWrap.
  • VideoPlayer (missing description).
  • AxiosForm.
  • CookieInformation (missing description).
  • CookieInformationBlocker (should be added).
  • HeadingX.
  • LinkTile.
  • NuxtLinkExt.
  • PostScribeScript.
  • StructuredData (missing description).
  • TransitionExt.
  • UmbracoForm (missing description).
  • UmbracoSvg (should be added).

Directives

  • Tab directive (should perhaps be added, at some point).
  • Trap focus directive (should perhaps be added, at some point).

Wrote documentation

See documentation here.

Added plop

.npmrc

Request GitHub PAT, and adds a new .npmrc file, which is required to, among other things, publish packages.

package

Requests type (vue or javascript), name and description, which is used to generate a new package.
Currently the JS-type package consists of an index.js file and a dev directory.
Currently the Vue-type package consists of src, dev and test directories.
The naming sort of gives their purpose away.

Discuss:

  • Should we have the possibility to add TypeScript packages - or just be really good at writing JSDocs comments, which there's actually a plugin for? (see here). Write something about the documentation - link to docs.
  • There's some general naming issues with JavaScript packages, fx. everything dev-related is called serve - taking directly from the Vue-packages - this should probably not be the case.

Other

There's currently five unresolved tasks in the mono-repo README, what's the status on these?
There's currently an unresolved pull-request from Saul for the screens package. Should this be merged?

In the pull-request described above, Fjordside also asks whether or not freelancers will be able to install packages.
The answer to which is yes, given that the packages aren't private. If they are, they might need to be added as collaborators.

@AskeLange AskeLange self-assigned this Oct 10, 2022
@AskeLange AskeLange changed the title Changes to the mono-repo From frontend-meet: Changes to the mono-repo Oct 10, 2022
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

No branches or pull requests

1 participant