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

Edit workflow confusion? #19

Open
nasso opened this issue Mar 23, 2024 · 1 comment
Open

Edit workflow confusion? #19

nasso opened this issue Mar 23, 2024 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@nasso
Copy link

nasso commented Mar 23, 2024

But sometimes, when we're working on something, we realize we also want a
different change, and maybe it relies on this one. For example, let's say that
we were working on undoing this goodbye feature, but we realized we wanted
to refactor printing out into its own function, because that's a terrible idea
in practice and so makes for a good example to play around with.

here you mention how we may want to "refactor printing out into its own function"

expected behaviour

the rest of the example goes through the steps of how we would do that refactoring

actual behaviour

we just add a comment

additional context

operating system: android
browser: firefox focus
theme: navy
rocket league rank: c1d1
effort put in writing this critical bug report: yes

(maybe you just didn't want to talk about conflicts at this time?)

@steveklabnik steveklabnik self-assigned this Mar 25, 2024
@steveklabnik steveklabnik added the bug Something isn't working label Mar 25, 2024
@steveklabnik
Copy link
Owner

(maybe you just didn't want to talk about conflicts at this time?)

Yeah, so originally I had some stuff with conflicts earlier than this, and then later, came back and moved that stuff to a later section. So this is an inconsistency.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants