Skip to content

Conversation

@ReenigneArcher
Copy link
Member

@ReenigneArcher ReenigneArcher commented Mar 23, 2025

Description

Add jsdoc, readthedocs, and examples.

Screenshot

Issues Fixed or Closed

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Dependency update (updates to dependencies)
  • Documentation update (changes to documentation)
  • Repository update (changes to repository files, e.g. .github/...)

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated the in code docstring/documentation-blocks for new or existing methods/components

@codecov
Copy link

codecov bot commented Mar 23, 2025

Bundle Report

Changes will increase total bundle size by 2.98kB (11.96%) ⬆️⚠️, exceeding the configured threshold of 5%.

Bundle name Size Change
shared-web-array-push 27.94kB 2.98kB (11.96%) ⬆️⚠️

Affected Assets, Files, and Routes:

view changes for bundle: shared-web-array-push

Assets Changed:

Asset Name Size Change Total Size Change (%)
crowdin.js -376 bytes 3.56kB -9.56%
crowdin-clean-jsdoc-css.css (New) 1.29kB 1.29kB 100.0% 🚀
crowdin-doxygen-css.css (New) 719 bytes 719 bytes 100.0% 🚀
crowdin-furo-css.css (New) 698 bytes 698 bytes 100.0% 🚀
crowdin-bootstrap-css.css (New) 653 bytes 653 bytes 100.0% 🚀

@codecov
Copy link

codecov bot commented Mar 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.27%. Comparing base (36dab7e) to head (4aec818).
Report is 16 commits behind head on master.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #101      +/-   ##
==========================================
- Coverage   98.37%   98.27%   -0.10%     
==========================================
  Files           8        8              
  Lines         123      116       -7     
  Branches       32       31       -1     
==========================================
- Hits          121      114       -7     
  Misses          2        2              
Files with missing lines Coverage Δ
src/js/crowdin.js 100.00% <100.00%> (ø)
src/js/discord.js 88.88% <ø> (ø)
src/js/format-number.js 100.00% <ø> (ø)
src/js/levenshtein-distance.js 100.00% <ø> (ø)
src/js/load-script.js 100.00% <ø> (ø)
src/js/ranking-sorter.js 100.00% <ø> (ø)
src/js/sleep.js 100.00% <ø> (ø)

@ReenigneArcher ReenigneArcher force-pushed the docs/add-jsdoc-and-rtd-build branch 28 times, most recently from ae9e6f2 to 374a514 Compare March 24, 2025 01:02
@ReenigneArcher ReenigneArcher force-pushed the docs/add-jsdoc-and-rtd-build branch 8 times, most recently from db8b7cf to 95db90a Compare March 25, 2025 01:54
@ReenigneArcher ReenigneArcher force-pushed the docs/add-jsdoc-and-rtd-build branch 13 times, most recently from b6d57f2 to 9d1ea28 Compare March 26, 2025 00:46
@ReenigneArcher ReenigneArcher force-pushed the docs/add-jsdoc-and-rtd-build branch from 9d1ea28 to 4aec818 Compare March 26, 2025 00:51
@ReenigneArcher ReenigneArcher marked this pull request as ready for review March 26, 2025 00:53
@ReenigneArcher ReenigneArcher requested a review from Copilot March 26, 2025 00:57
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds documentation build configurations, automated workflows, and sample projects to support the shared-web documentation and examples using jsdoc, ReadTheDocs, Jekyll, Doxygen, and release automation.

  • Introduces a new ReadTheDocs configuration with multiple build commands.
  • Adds example projects for Jekyll and Doxygen along with associated configurations and asset initialization.
  • Updates GitHub workflows for docs and release management.

Reviewed Changes

Copilot reviewed 37 out of 47 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.readthedocs.yaml New RTD configuration with build commands for various documentation tools.
examples/jekyll/shared-web-jekyll.gemspec Adds a gemspec file for the Jekyll example project.
examples/doxygen/sample.md Provides a Doxygen sample project demonstrating widget integration.
.github/workflows/update-docs.yml Introduces a new workflow for updating ReadTheDocs projects on release events.
examples/jekyll/_config.yml Supplies Jekyll configuration for the sample site with custom theme settings.
examples/jekyll/Gemfile Minimal Gemfile for the Jekyll example project.
examples/jekyll/index.md A landing page markdown file for the Jekyll sample project.
examples/jekyll/assets/js/crowdin-init.js JS file initializing the CrowdIn integration.
.github/workflows/ci.yml Extends CI with a new release job invoking a custom create-release action.
examples/jekyll/assets/js/discord-init.js JS file initializing Discord integration.
Files not reviewed (10)
  • .flake8: Language not supported
  • examples/doxygen/Doxyfile: Language not supported
  • examples/doxygen/header.html: Language not supported
  • examples/doxygen/package.json: Language not supported
  • examples/jekyll/package.json: Language not supported
  • examples/sphinx/.rstcheck.cfg: Language not supported
  • examples/sphinx/Makefile: Language not supported
  • examples/sphinx/make.bat: Language not supported
  • examples/sphinx/package.json: Language not supported
  • examples/sphinx/requirements.txt: Language not supported
Comments suppressed due to low confidence (1)

examples/jekyll/shared-web-jekyll.gemspec:6

  • [nitpick] The author name 'ReenigneArcher' might be misspelled; please verify if this is the intended identifier.
spec.authors       = ["ReenigneArcher"]

@ReenigneArcher ReenigneArcher merged commit b2571ce into master Mar 26, 2025
14 checks passed
@ReenigneArcher ReenigneArcher deleted the docs/add-jsdoc-and-rtd-build branch March 26, 2025 01:01
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