Skip to content

Conversation

@DanielVoogsgerd
Copy link
Collaborator

@DanielVoogsgerd DanielVoogsgerd commented Mar 19, 2025

Still a work in progress

To Do:

  • Pass tests again
  • Implement proper testing

}
}

// TODO: These impls probably need to go somewhere

Check notice

Code scanning / devskim

A "TODO" or similar was left in source code, possibly indicating incomplete functionality Note

Suspicious comment
fn take(&self, count: usize) -> Self { Tokens { tok: &self.tok[0..count], start: 0, end: count } }

#[inline]
// TODO: This seems new

Check notice

Code scanning / devskim

A "TODO" or similar was left in source code, possibly indicating incomplete functionality Note

Suspicious comment
@DanielVoogsgerd DanielVoogsgerd added A-DSL Area: Something that relates to one of Brane's domain-specific languages (BraneScript / Bakery) C-Dependencies Category: Changes that *only* affect dependencies C-Maintenance Category: Maintenance E-Hard Effort: Hard P-Medium Priority: Medium labels Mar 19, 2025
@codecov
Copy link

codecov bot commented Mar 19, 2025

Also moved these integration tests to the correct location for
integration tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-DSL Area: Something that relates to one of Brane's domain-specific languages (BraneScript / Bakery) C-Dependencies Category: Changes that *only* affect dependencies C-Maintenance Category: Maintenance E-Hard Effort: Hard P-Medium Priority: Medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants