Skip to content

fix: simplify linkup import with aliases and some cleanup#70

Merged
cjumel merged 3 commits into
mainfrom
cj/maintenance
May 21, 2026
Merged

fix: simplify linkup import with aliases and some cleanup#70
cjumel merged 3 commits into
mainfrom
cj/maintenance

Conversation

@cjumel
Copy link
Copy Markdown
Contributor

@cjumel cjumel commented May 20, 2026

Description

  • chore: replace mdformat by rumdl
    This follows our internal convention. Rumdl is faster and has more features (like dealing well with front-matters) than mdformat.
  • chore: remove examples
    We decided to move most of the documentation in our API documentation, so let's only keep the examples mentioned in the README.
  • fix: simplify linkup import with aliases
    This will make possible to import linkup with import linkup and use it with linkup.Client() which is more concise and follows the google style guide recommendation for imports, which is to only import modules. Former import style is left and will be kept in the foreseeable future.

Checklist

  • I have installed prek on this project (for instance with the make install-dev command)
    before creating any commit, or I have run successfully the make lint command on my changes.
  • I have run successfully the make typecheck test command on my changes.
  • I have updated the README.md if my changes affected it.

cjumel added 2 commits May 20, 2026 18:19
This follows our internal convention. Rumdl is faster and has more
features (like dealing well with front-matters) than mdformat.
We decided to move most of the documentation in our API documentation,
so let's only keep the examples mentioned in the README.
@cjumel cjumel requested a review from Shedna May 20, 2026 16:26
@cjumel cjumel changed the title cj/maintenance fix: nullable favicon and simpler imports May 20, 2026
@cjumel cjumel marked this pull request as ready for review May 20, 2026 16:26
This will make possible to import linkup with `import linkup` and use it
with `linkup.Client()` which is more concise and follows the google
style guide recommendation for imports, which is to only import modules.

Former import style is left and will be kept in the foreseeable future.
@cjumel cjumel changed the title fix: nullable favicon and simpler imports fix: simplify linkup import with aliases and some cleanup May 21, 2026
@cjumel cjumel merged commit ec20202 into main May 21, 2026
3 checks passed
@cjumel cjumel deleted the cj/maintenance branch May 21, 2026 08:54
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