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

Added fromObject and copy methods to all Two.Elements and descendants #741

Merged
merged 24 commits into from
Nov 7, 2024

Conversation

jonobr1
Copy link
Owner

@jonobr1 jonobr1 commented Nov 7, 2024

@jonobr1 jonobr1 merged commit 1b3a45b into dev Nov 7, 2024
1 check passed
@jonobr1 jonobr1 deleted the 737-from-object branch November 7, 2024 06:17
static fromObject(obj) {
let stops = obj.stops;
if (stops && stops.length > 0) {
stops = stops.map((o) => (o instanceof Stop ? o : new Stop().copy(o)));

Check warning

Code scanning / CodeQL

Useless assignment to local variable Warning

The value assigned to stops here is unused.
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.

1 participant