Reviewer names - #73
Merged
Merged
Conversation
jchiquet
approved these changes
Jul 7, 2026
fradav
pushed a commit
to computorg/computorg-new.github.io
that referenced
this pull request
Jul 26, 2026
* update paragraph on reviewers' anonymity computorg#70 * add reviewer names
fradav
added a commit
that referenced
this pull request
Jul 26, 2026
* - Refactoring of repositories parsing metadata to use `quarto inspect` logic instead of manual metadata extraction/parsing. - Removed a small redundant subsection tittle in the authors guide. - RSS feed generated by the main publication script (as quarto is not capable of generating RSS feeds directly from an existing yml). * Update build workflow to stage both YAML and XML published files * Update publications from getcomputo-pub.fsx [skip ci] * 2026 may update * forgot dotnet tool * Fix workflow indentation and build command Steps are now properly nested under the jobs definition. Remove the UpdatePublications target argument from the refresh step. Track .config/dotnet-tools.json by updating gitignore. * Revert * Refresh publication metadata [skip ci] * Update dependency chain in Build.fs Remove UpdatePublications target from default chain * Update lab affiliation for Fradav * Borked link to pci * Fix article pages missing from sitemap causing indexing issues Google Search Console diagnostics showed that published article pages (hosted from separate computorg/published-* repos, served as subpaths of computo-journal.org) were mostly stuck in "Crawled - currently not indexed", while the site's sitemap.xml only listed the 17 core pages and never referenced any of the 21 published articles. - Normalize article URLs to https in PublicationUpdater.fs and in the already-committed site/published.yml (13 entries were http://, producing duplicate-content signals since those repos didn't enforce HTTPS) - Generate a sitemap fragment (site/published-sitemap-fragment.xml) listing all published article URLs whenever publications are refreshed - Merge that fragment into _site/sitemap.xml after quarto render in Build.fs, so the deployed sitemap now includes article pages alongside the core site pages Also enabled "Enforce HTTPS" on the 14 computorg/published-* repos where it was disabled, via the GitHub API (separate from this repo, not part of this commit). * Remove old publication updater and update CI checkout Remove all F# source files, tests, and Paket dependency management for the publication updater. Update the CI workflow to use actions/checkout@v4 and clarify the checkout step name. * fix: configure NuGet source with credentials before installing dotnet tool * fix: remove duplicate NuGet source addition, use already-configured source * fix: use API_GITHUB_TOKEN for NuGet source authentication * fix: use API_GITHUB_TOKEN for NuGet source auth in install step * Refresh publication metadata [skip ci] * Refresh publication metadata [skip ci] * Refresh publication metadata [skip ci] * Refresh publication metadata [skip ci] * Refresh publication metadata [skip ci] * Add TODO.md and workflow dispatch force option for build.yml * Reviewer names (#73) * update paragraph on reviewers' anonymity #70 * add reviewer names * Guidelines authors (#71) * updates for #69: remove earlier links to templates and rename to main.qmd * Address most of #69 * rm section "Accessing source code" (the recommended way to access the source code is documented in the FAQ) * update section on python dependency management * fix formatting of reviewer names * more work on alphabetic order * feat: add Open Graph and social metadata (#72) Enable `open-graph` and `twitter-card` in Quarto and add a branded 1200x630 social card so links render with an image on social media. Replace the homepage's HTML `description` (metadata-only on this page) with a plain sentence so the meta description no longer repeats the title. Also improve indexing: set `canonical-url`, drop the trailing slash from `site-url` for clean absolute URLs, and add a post-render hook to align sitemap entries with the canonical trailing-slash form. Fixes #65 * Refresh publication metadata [skip ci] * Refresh publication metadata [skip ci] * Refresh publication metadata [skip ci] * Cleaning * Removed unused xml * Removed TODO.md --------- Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Julien Chiquet <julien.chiquet@inrae.fr> Co-authored-by: Pierre Neuvial <pneuvial@users.noreply.github.com> Co-authored-by: Pierre Neuvial <pierre.neuvial@math.cnrs.fr> Co-authored-by: Johan Larsson <13087841+jolars@users.noreply.github.com>
fradav
added a commit
that referenced
this pull request
Jul 26, 2026
* - Refactoring of repositories parsing metadata to use `quarto inspect` logic instead of manual metadata extraction/parsing. - Removed a small redundant subsection tittle in the authors guide. - RSS feed generated by the main publication script (as quarto is not capable of generating RSS feeds directly from an existing yml). * Update build workflow to stage both YAML and XML published files * Update publications from getcomputo-pub.fsx [skip ci] * 2026 may update * forgot dotnet tool * Fix workflow indentation and build command Steps are now properly nested under the jobs definition. Remove the UpdatePublications target argument from the refresh step. Track .config/dotnet-tools.json by updating gitignore. * Revert * Refresh publication metadata [skip ci] * Update dependency chain in Build.fs Remove UpdatePublications target from default chain * Update lab affiliation for Fradav * Borked link to pci * Fix article pages missing from sitemap causing indexing issues Google Search Console diagnostics showed that published article pages (hosted from separate computorg/published-* repos, served as subpaths of computo-journal.org) were mostly stuck in "Crawled - currently not indexed", while the site's sitemap.xml only listed the 17 core pages and never referenced any of the 21 published articles. - Normalize article URLs to https in PublicationUpdater.fs and in the already-committed site/published.yml (13 entries were http://, producing duplicate-content signals since those repos didn't enforce HTTPS) - Generate a sitemap fragment (site/published-sitemap-fragment.xml) listing all published article URLs whenever publications are refreshed - Merge that fragment into _site/sitemap.xml after quarto render in Build.fs, so the deployed sitemap now includes article pages alongside the core site pages Also enabled "Enforce HTTPS" on the 14 computorg/published-* repos where it was disabled, via the GitHub API (separate from this repo, not part of this commit). * Remove old publication updater and update CI checkout Remove all F# source files, tests, and Paket dependency management for the publication updater. Update the CI workflow to use actions/checkout@v4 and clarify the checkout step name. * fix: configure NuGet source with credentials before installing dotnet tool * fix: remove duplicate NuGet source addition, use already-configured source * fix: use API_GITHUB_TOKEN for NuGet source authentication * fix: use API_GITHUB_TOKEN for NuGet source auth in install step * Refresh publication metadata [skip ci] * Refresh publication metadata [skip ci] * Refresh publication metadata [skip ci] * Refresh publication metadata [skip ci] * Refresh publication metadata [skip ci] * Add TODO.md and workflow dispatch force option for build.yml * Reviewer names (#73) * update paragraph on reviewers' anonymity #70 * add reviewer names * Guidelines authors (#71) * updates for #69: remove earlier links to templates and rename to main.qmd * Address most of #69 * rm section "Accessing source code" (the recommended way to access the source code is documented in the FAQ) * update section on python dependency management * fix formatting of reviewer names * more work on alphabetic order * feat: add Open Graph and social metadata (#72) Enable `open-graph` and `twitter-card` in Quarto and add a branded 1200x630 social card so links render with an image on social media. Replace the homepage's HTML `description` (metadata-only on this page) with a plain sentence so the meta description no longer repeats the title. Also improve indexing: set `canonical-url`, drop the trailing slash from `site-url` for clean absolute URLs, and add a post-render hook to align sitemap entries with the canonical trailing-slash form. Fixes #65 * Refresh publication metadata [skip ci] * Refresh publication metadata [skip ci] * Refresh publication metadata [skip ci] * Cleaning * Removed unused xml * Removed TODO.md * Restore FAQ entry and lua script (without documentation note) - blog/2026-07-26-google-scholar/index.qmd: FAQ entry for Google Scholar - scripts/fix-sitemap.lua: Lua replacement for bash sitemap script - _quarto.yml: Updated to include FAQ in navigation - .gitignore: Updated ignores - Removed scripts/fix-sitemap.sh (replaced by lua) --------- Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Julien Chiquet <julien.chiquet@inrae.fr> Co-authored-by: Pierre Neuvial <pneuvial@users.noreply.github.com> Co-authored-by: Pierre Neuvial <pierre.neuvial@math.cnrs.fr> Co-authored-by: Johan Larsson <13087841+jolars@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add a list of reviewers (#70)