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

Update AI documentation #497

Merged
merged 14 commits into from
Aug 13, 2024
Merged

Update AI documentation #497

merged 14 commits into from
Aug 13, 2024

Conversation

InAnYan
Copy link
Contributor

@InAnYan InAnYan commented Aug 13, 2024

I think the new documentation is not deployed right now?

I also found a SUMMARY.md file, where there was no AI. So I made a separate PR for this

@koppor
Copy link
Member

koppor commented Aug 13, 2024

This is a follow-up to #491.

I do understand why here are so many changes. Need to look into it later.

@koppor
Copy link
Member

koppor commented Aug 13, 2024

I found the issue:

  1. Add AI documentation #491 was squash-merged
  2. This PR was based on the branch of Add AI documentation #491
  3. A new commit ebcc0e1 (#497) was added

"Just" merging main fixes the issue of GitHub showing too much difference. - Always starting from main is also a workaround of the issue.

@koppor
Copy link
Member

koppor commented Aug 13, 2024

We do a bad job with our CONTRIBUTING guidelines.

Maybe you can help to move content from https://github.com/JabRef/user-documentation?tab=readme-ov-file#advanced-contribution-hints to https://github.com/JabRef/user-documentation/blob/main/en/contributing/how-to-improve-the-help-page.md. -- I think, it is "just" text moval?

@koppor koppor merged commit a89d8c9 into JabRef:main Aug 13, 2024
3 checks passed
@koppor koppor deleted the ai-1 branch August 13, 2024 13:38
@InAnYan
Copy link
Contributor Author

InAnYan commented Aug 14, 2024

Oliver, remember you told, that "Students don't think"

Student's (me) don't think :)

@InAnYan
Copy link
Contributor Author

InAnYan commented Aug 14, 2024

Ah, a squash-merge, I see. Sorry

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