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

Add command to install HumHub #650

Merged
merged 3 commits into from
Aug 26, 2024
Merged

Add command to install HumHub #650

merged 3 commits into from
Aug 26, 2024

Conversation

niristius
Copy link
Contributor

So I deleted the old branch since it was kinda stale and I am still trying to understand Node, yarn and oclif a bit more.
And since the changes are rather trivial I started over.

Sadly the yarn generate:readme command still adds newlines where they dont belong.

I tried

  • using Node 18, 20, 22
  • using oclif 4.14, 4.13, 4.12, 4.10, 3.17, 3.16, 3.15, 3.14

still no cigar.
@martin-helmich please advise

@niristius niristius changed the title Draft add humhub Draft: Add HumHub Jul 11, 2024
@tillwestpfahl
Copy link
Contributor

I have the same exact problem (see here). Also for some reason, the generated readme looks entirely different after a fresh repo clone.

@martin-helmich
Copy link
Member

I think the README issue should be resolved with #716; furthermore, the README should now be updated automatically in the PR if its not up-to-date.

Apart from the README, would this PR be ready for review?

@niristius
Copy link
Contributor Author

Yes, the readme was the only blocker for this PR

@martin-helmich martin-helmich changed the title Draft: Add HumHub Add command to install HumHub Aug 26, 2024
@martin-helmich martin-helmich merged commit 38b44c0 into master Aug 26, 2024
8 checks passed
@martin-helmich martin-helmich deleted the feat/add-humhub branch August 26, 2024 06:33
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