Skip to content
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

Fix indent in AlgebraicCycles.md #4368

Merged
merged 2 commits into from
Nov 29, 2024
Merged

Conversation

paemurru
Copy link
Contributor

@benlorenz
Copy link
Member

benlorenz commented Nov 29, 2024

Have you checked this locally? Unfortunately with PRs from a fork the CI cannot provide a preview.
Strictly speaking markdown lists should have: 2 spaces in front of the asterisk, one after and subsequent lines should be indented by 4 spaces.

@lgoettgens lgoettgens added the documentation Improvements or additions to documentation label Nov 29, 2024
@paemurru
Copy link
Contributor Author

I checked locally, it looked nice:

Screenshot from 2024-11-29 15-40-03

@benlorenz benlorenz enabled auto-merge (squash) November 29, 2024 15:27
Copy link

codecov bot commented Nov 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.34%. Comparing base (f83a525) to head (0e7be24).
Report is 208 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4368   +/-   ##
=======================================
  Coverage   84.34%   84.34%           
=======================================
  Files         651      651           
  Lines       86746    86746           
=======================================
  Hits        73167    73167           
  Misses      13579    13579           

@benlorenz benlorenz merged commit 278b79d into oscar-system:master Nov 29, 2024
30 checks passed
@paemurru paemurru deleted the patch-1 branch November 29, 2024 16:12
@lgoettgens lgoettgens mentioned this pull request Dec 12, 2024
benlorenz pushed a commit that referenced this pull request Dec 12, 2024
* Fix indent in AlgebraicCycles.md

It looks correct on GitHub but is incorrect in https://docs.oscar-system.org/dev/AlgebraicGeometry/ToricVarieties/AlgebraicCycles/

* Update AlgebraicCycles.md

(cherry picked from commit 278b79d)
benlorenz added a commit that referenced this pull request Dec 13, 2024
Backports for release 1.2.2:

fix galois_group problem. #4396
Fix zero-dimensional cone in cones in PolyhedralGeometry #4336
Fix up generic characteristic method for localized rings. #4346
test/PolyhedralGeometry: remove some prefer blocks #4351
Add comment about bibtool version, minor fix in testsuite #4356
Add hash method for RayVector #4354
Fix indent in AlgebraicCycles.md #4368
Fix toric typo #4367
@lgoettgens lgoettgens added the release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes label Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants