Docs stage 2

[DanielAvdar]

Uncommented the deployment step in the CI configuration to activate it. This step uses the sphinx-notes/pages action for documentation deployment.

[Copilot]

Copilot reviewed 5 out of 7 changed files in this pull request and generated no comments.

Files not reviewed (2)

• .github/workflows/iso_test.yml: Language not supported
• .readthedocs.yaml: Language not supported

Comments suppressed due to low confidence (3)

main.py:1

• The removal of the main block appears unrelated to the documentation deployment changes; please confirm that this removal is intentional.

if __name__ == "__main__":

.github/workflows/parsing-checks.yml:48

• Using 'continue-on-error: true' in the isolation check job might hide potential issues; please verify that this behavior is intended for your CI process.

continue-on-error: true

.github/workflows/ci.yml:36

• Consider removing the commented out deployment step to clean up the CI configuration if it is no longer needed.

#-      - id: deployment

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (f9463ad) to head (8e035ef).
Report is 3 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #144   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           12        12           
  Lines          330       330           
=========================================
  Hits           330       330           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Copilot reviewed 13 out of 22 changed files in this pull request and generated no comments.

Files not reviewed (9)

• .github/workflows/iso_test.yml: Language not supported
• .readthedocs.yaml: Language not supported
• docs/source/api_reference.rst: Language not supported
• docs/source/components.rst: Language not supported
• docs/source/contributing.rst: Language not supported
• docs/source/environment.rst: Language not supported
• docs/source/index.rst: Language not supported
• docs/source/introduction.rst: Language not supported
• docs/source/usage.rst: Language not supported

Comments suppressed due to low confidence (3)

tests/test_operation.py:42

• [nitpick] The test is directly invoking the private method '_create_kfp_str', which can lead to fragile tests if internal implementations change. Consider using the public API if one is available.

assert str_func == op._create_kfp_str(component=component1)

tests/test_dummies/test_write_to_file.py:11

• [nitpick] The test now calls the private method '_write_to_file', exposing internal implementation details. It is advisable to test functionality through the public interface to ensure stability.

fp._write_to_file(file_path, content)

docs/source/conf.py:16

• [nitpick] The variable 'version' shadows the imported function 'version' from importlib.metadata. Consider renaming the variable (e.g., 'project_version') to avoid confusion.

version = version(project)