Chewing_Bever
/
fej
1
0
Fork 0
Code Issues 28 Pull Requests 10 Projects 3 Releases 4 Wiki Activity

Ivago: replace String with &str wherever possible #7

New Issue
Closed
opened 2021-04-04 12:08:25 +02:00 by Jef Roosens · 1 comment
Jef Roosens commented 2021-04-04 12:08:25 +02:00
Owner

Considering that we don't actually need any string manipulations, we can replace String in a lot of places, hopefully simplifying the codebase a little bit.

Considering that we don't actually need any string manipulations, we can replace String in a lot of places, hopefully simplifying the codebase a little bit.
Jef Roosens added this to the Version 1.0: Ivago milestone 2021-04-04 12:08:25 +02:00
Jef Roosens added the
enhancement
 label 2021-04-04 12:08:25 +02:00
Jef Roosens self-assigned this 2021-04-04 12:08:25 +02:00
Jef Roosens added this to the (deleted) project 2021-04-04 12:08:25 +02:00
Jef Roosens commented 2021-04-04 14:33:38 +02:00
Poster
Owner

I'm not actually sure if this has a point, gonna need to do some more research.

I'm not actually sure if this has a point, gonna need to do some more research.
Jef Roosens removed this from the (deleted) project 2021-04-05 10:01:05 +02:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
renovate/reqwest-0.x
renovate/serde-1.x
renovate/eslint-plugin-vue-8.x
renovate/vitejs-plugin-vue-2.x
renovate/typescript-eslint-parser-5.x
renovate/typescript-eslint-eslint-plugin-5.x
renovate/vue-tsc-0.x
renovate/chrono-tz-0.x
develop
renovate/eslint-plugin-promise-6.x
renovate/alpine-3.x
68-clippy
master
v1.0.2
v1.0.1
v1.0.0
v0.1.0
Labels
Clear labels   
Brainstorm

Place to brainstorm problems or possible ideas   
Parent

Description of larger problem with multiple issues   
bug

Something is not working   
duplicate

This issue or pull request already exists   
enhancement

New feature   
help wanted

Need some help   
idea

A possible idea for a feature   
invalid

Something is wrong   
project/backend

An issue related to the backend   
project/frontend

An issue related to the frontend   
question

More information is needed   
wontfix

This won't be fixed
No Label
Brainstorm
Parent
bug
duplicate
enhancement
help wanted
idea
invalid
project/backend
project/frontend
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Version 1.0: Ivago
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
Jef Roosens
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Chewing_Bever/fej#7
There is no content yet.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?