Feat/ga checks

[CIGbalance]

Adds the GitHub Action responsible for triggering the validation of the new problem

Note that this check is only supposed to be triggered when the template file "new_problem.yaml" is changed. It will mark a failure if the name is not changed, that's why the test currently fails here.

Closes #124

[kvdblom]

Very small change, otherwise looks good!

[kvdblom]

Looks good, only merge after: #114

[Copilot]

Pull request overview

This PR introduces a GitHub Actions workflow intended to validate updates to utils/new_problem.yaml (consistency/novelty checks), and additionally updates the YAML→HTML generation pipeline and regenerated docs/content.

Changes:

• Add New Problem Check GitHub Action to run utils/validate_yaml.py when utils/new_problem.yaml changes.
• Update yaml_to_html.py to linkify URLs in cells and adjust HTML rendering options; regenerate docs HTML accordingly.
• Add multiple new entries to problems.yaml and update contribution instructions (README + site header/index) to include a Google Form.

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
.github/workflows/new_problem_check.yml	New workflow to validate utils/new_problem.yaml on PRs/pushes.
yaml_to_html.py	Adds URL linkification and changes HTML escaping/link rendering behavior.
utils/new_problem.yaml	Template edited (currently triggers failing validation).
problems.yaml	Appends many new problems/suites (some data quality issues found).
docs/problems.html	Regenerated table output reflecting new linkification + new problems.
docs/index.html	Updates submission text + regenerated table output + DataTables config change.
docs/header.html	Adds Google Form submission option.
docs/javascript.html	Enables DataTables search widget in the layout.
README.md	Adds Google Form link for submissions.
.gitignore	Expanded ignore rules (Python-focused template).

Comments suppressed due to low confidence (1)

yaml_to_html.py:34

• This file executes its YAML→HTML generation at import time (top-level code). Since utils/validate_yaml.py imports default_columns from this module, running the validation workflow will also regenerate docs and require pandas, which is an unexpected side effect for a validator. Consider moving the generation logic into a main() guarded by if __name__ == "__main__":, and keep only constants/helpers importable.

yaml_file = "problems.yaml"

html_dir = "docs/"
html_table = f"{html_dir}problems.html"
html_header = f"{html_dir}header.html"
html_scripts = f"{html_dir}javascript.html"
html_footer = f"{html_dir}footer.html"
html_index = f"{html_dir}index.html"

# Load data
with open(yaml_file) as in_file:
    data = pd.json_normalize(yaml.safe_load(in_file))

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

escape=False combined with inserting raw <a> tags means any HTML present in problems.yaml will be rendered directly into the generated docs, which is an XSS/vector for script injection (and also produces invalid HTML for URLs containing & since they won't be escaped to &). Safer options: keep HTML escaping enabled and post-process the generated table HTML to linkify URLs, or ensure linkify_cell HTML-escapes non-URL text and properly escapes attribute values (e.g., &, <, >).

[Copilot]

As written, this change will make the new workflow fail: utils/validate_yaml.py errors if name is still template, and this PR modifies utils/new_problem.yaml without changing the name. Either revert this template edit before merging, or update the entry to a non-template name so the check passes in this PR.

Suggested change

	- name: template
	- name: new_problem

[Copilot]

The name value has leading whitespace (" VehicleDynamics"), which will be treated as part of the name string in YAML and can break uniqueness checks / lookups (and render inconsistently). Please remove the extra spaces so the name is exactly VehicleDynamics.

Suggested change

	- name: VehicleDynamics
	- name: VehicleDynamics

[Copilot]

dimensionality: scalable' includes a stray trailing quote. This will propagate into the generated HTML (and any downstream processing) as scalable' rather than scalable. Please correct the value.

Suggested change

	dimensionality: scalable'
	dimensionality: scalable

[Copilot]

actions/checkout@v3 and actions/setup-python@v4 are older major versions; GitHub periodically deprecates old majors. Consider upgrading to the current majors and pinning the Python version to a specific minor (instead of 3.x) for more reproducible validation behavior.

Suggested change

	uses: actions/checkout@v3
	- name: Set up Python
	uses: actions/setup-python@v4
	with:
	python-version: '3.x'
	uses: actions/checkout@v4
	- name: Set up Python
	uses: actions/setup-python@v5
	with:
	python-version: '3.12'

[Copilot]

The PR description is focused on adding a GitHub Action for validating utils/new_problem.yaml, but this PR also includes unrelated changes (URL linkification logic, docs HTML regeneration, README updates, and a large problems.yaml content addition). If possible, split these into separate PRs or update the PR description to reflect the additional scope so reviewers know what to validate.

[Copilot]

Line has trailing whitespace after the period. Please remove it to avoid unnecessary diffs and keep Markdown formatting clean.

Suggested change

	Contributions and corrections are very welcome, through pull requests, issue reporting, or email.
	Contributions and corrections are very welcome, through pull requests, issue reporting, or email.