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

doc: Added yaml-metadata doc into hedgedoc frontend #4371

Merged
merged 1 commit into from Jul 10, 2023

Conversation

JunedKhan101
Copy link
Contributor

Component/Part

Documentation

Description

This PR add yaml-metadata documentation to hedgedoc documentaion from public/docs.

Steps

  • Added / updated documentation
  • Added changelog snippet
  • I read the contribution documentation and
    made sure that:
    • My commits are signed-off to accept the DCO.
    • This PR targets the correct branch: master for 1.x & docs, develop for 2.x

Related Issue(s)

#1427

public/docs/release-notes.md Outdated Show resolved Hide resolved
docs/content/references/yaml-metadata.md Outdated Show resolved Hide resolved
@JunedKhan101
Copy link
Contributor Author

I have added commits that addresses the requested changes. Please check @mrdrogdrog @ErikMichelson

Copy link
Member

@ErikMichelson ErikMichelson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Please squash your two commits into one (the one starting with doc: should be kept) and rebase onto latest master branch to avoid merge commits.

@JunedKhan101
Copy link
Contributor Author

Hi @ErikMichelson I have squashed my commits but I don't quite understand which branch should be rebased into which branch.

On my docs-yaml-metadata local branch I tried the command:
git rebase develop

However, I ran into merge conflicts

Could you please guide me through this as I am unfamiliar with rebase.

@mrdrogdrog
Copy link
Member

mrdrogdrog commented Jul 5, 2023

Don't rebase on the develop branch if you're creating a PR for master.

@JunedKhan101
Copy link
Contributor Author

JunedKhan101 commented Jul 5, 2023

@mrdrogdrog Does it look good now? Or have I done something else that was unnecessary?
It said successfully rebased to master so i think it worked please verify.

Signed-off-by: Juned Khan <junedkhanc101@gmail.com>
@JunedKhan101
Copy link
Contributor Author

JunedKhan101 commented Jul 10, 2023

@ErikMichelson An approval is required from your end to merge this.

@ErikMichelson ErikMichelson merged commit ceab037 into hedgedoc:master Jul 10, 2023
5 checks passed
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.

None yet

3 participants