Skip to content

Biblio integration #7

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

Closed
wants to merge 9 commits into from
Closed

Conversation

loumir
Copy link
Collaborator

@loumir loumir commented May 5, 2025

  • add obscure.bib for ref not in docrepo.bib
  • add \citep tags for all ref in docrepo.bib and obscore.bib

loumir added 6 commits May 5, 2025 17:40
inserting ADS id in text , compliant to docrepo.bib
part 1
biblio reference update
for all citations not in docrepo.bib
added \bibliography{ivoatex/docrepo, obscure}
\bibliography{ivoatex/docrepo, obscore}
@loumir
Copy link
Collaborator Author

loumir commented May 5, 2025

I think these update deal only with citations and bibliography .
It should not interfere with the figures updates in the recent pull request .

@pdowler
Copy link
Contributor

pdowler commented May 5, 2025

I merged the diagrams PR and fixed the two conflicts that created (that will be a commit in your branch). I also added write permission and approved the workflow for this PR, but the build is failing... not sure if it is previous work or these changes.

@loumir
Copy link
Collaborator Author

loumir commented May 6, 2025

OK , I will do it separately in a local repo via the command line .

@pdowler
Copy link
Contributor

pdowler commented May 8, 2025

Now that I reviewed and merged PR #8 is this now obsolete? Look like but I want to confirm... if so, please close this one.

@loumir
Copy link
Collaborator Author

loumir commented May 8, 2025

it seems all changes dealing with have been inserted in PR#8 . thanks

@loumir loumir closed this May 8, 2025
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