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

Add remarks to live/lb/position.md #24

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

XertroV
Copy link

@XertroV XertroV commented Dec 21, 2024

Remarks: duplicate mapUids not supported, and scores must be >= 0.

Remarks: duplicate mapUids not supported, and scores must be >= 0.
@XertroV XertroV requested a review from davidbmaier December 21, 2024 08:08
@codecat
Copy link
Member

codecat commented Dec 21, 2024

The duplicate mapUid thing feels a bit like an obvious remark that probably doesn't need to be be this high in the list of remarks, whereas the minimum score of 1 is a lot more important. Can you move the line with the minimum score above the duplicate mapUid one?

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