Skip to content

[Merged by Bors] - Correctly parse labels with '#'#5729

Closed
merelymyself wants to merge 1 commit intobevyengine:mainfrom
merelymyself:patch-1
Closed

[Merged by Bors] - Correctly parse labels with '#'#5729
merelymyself wants to merge 1 commit intobevyengine:mainfrom
merelymyself:patch-1

Conversation

@merelymyself
Copy link
Contributor

Objective

Solution

  • Used splitn instead of split to collect the rest of the string into the label after the first '#'.

@mockersf mockersf added C-Bug An unexpected or incorrect behavior A-Assets Load files from disk to use for things like images, models, and sounds labels Aug 18, 2022
@mockersf mockersf added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Aug 18, 2022
@cart
Copy link
Member

cart commented Aug 18, 2022

bors r+

bors bot pushed a commit that referenced this pull request Aug 18, 2022
# Objective

- Fixes #5707 

## Solution

- Used `splitn` instead of `split` to collect the rest of the string into the label after the first '#'.

---
@bors
Copy link
Contributor

bors bot commented Aug 18, 2022

@bors bors bot changed the title Correctly parse labels with '#' [Merged by Bors] - Correctly parse labels with '#' Aug 18, 2022
@bors bors bot closed this Aug 18, 2022
inodentry pushed a commit to BroovyJammy/bevy that referenced this pull request Aug 19, 2022
# Objective

- Fixes bevyengine#5707 

## Solution

- Used `splitn` instead of `split` to collect the rest of the string into the label after the first '#'.

---
maccesch pushed a commit to Synphonyte/bevy that referenced this pull request Sep 28, 2022
# Objective

- Fixes bevyengine#5707 

## Solution

- Used `splitn` instead of `split` to collect the rest of the string into the label after the first '#'.

---
james7132 pushed a commit to james7132/bevy that referenced this pull request Oct 28, 2022
# Objective

- Fixes bevyengine#5707 

## Solution

- Used `splitn` instead of `split` to collect the rest of the string into the label after the first '#'.

---
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
# Objective

- Fixes bevyengine#5707 

## Solution

- Used `splitn` instead of `split` to collect the rest of the string into the label after the first '#'.

---
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Assets Load files from disk to use for things like images, models, and sounds C-Bug An unexpected or incorrect behavior S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Asset path wrong parsed (probably)

4 participants