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

Article Submission #426

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Article Submission #426

wants to merge 15 commits into from

Conversation

ology
Copy link
Collaborator

@ology ology commented Mar 15, 2025

Maybe this will work on a different branch? IDK...

@oalders
Copy link
Contributor

oalders commented Mar 15, 2025

I pushed your branch to this repo and created a PR based on that. You can see it at #427. You can push to that branch if you like. Whatever works for you. At the very least, we'll have a preview shortly. 😄

@ology
Copy link
Collaborator Author

ology commented Mar 15, 2025

@oalders Thank you

@ology
Copy link
Collaborator Author

ology commented Mar 16, 2025

@oalders I apologize for not following sane practices for naming my branch. It should have been something like ology/enhancing... :\

@oalders
Copy link
Contributor

oalders commented Mar 16, 2025

I'm just happy to get more quality content!

@ology
Copy link
Collaborator Author

ology commented Mar 16, 2025

I pushed to my ugly-named branch, but don't see the commit in either of the #426 or #427 PRs. I'm afraid I've done something wrong, yet again. :\

@ology
Copy link
Collaborator Author

ology commented Mar 16, 2025

No I see it! "Section" - Just hasn't been deployed I guess.

@jbarrett
Copy link
Contributor

jbarrett commented Mar 16, 2025

Hey folks, just a heads-up - it's a public holiday here at the moment, so I'm taking things a bit easier than usual.

I'll start on this by Wednesday, if that fits with the timelines. Cheers!

@ology
Copy link
Collaborator Author

ology commented Mar 17, 2025

Not sure why, but the trial deployment link above, https://www-perl-com-pr-427.onrender.com/article/enhancing-your-midi-devices-round-ii/ is an old version. Not sure if I need to click a "Deploy!" link somewhere. Anyway, local hugo server --buildDrafts --buildFuture works! :D

@oalders
Copy link
Contributor

oalders commented Mar 17, 2025

The trial link is based off of a branch that I pushed to perladvent/perldotcom and you're currently pushing to https://github.com/ology/perldotcom/tree/EYMDRII, so the deployment is not aware of the changes. Not a big deal, though.

@ology
Copy link
Collaborator Author

ology commented Mar 17, 2025

I am a dork. I can't believe I was employed to do technical things for as long as I was. (But unemployed now. Yay!)

@oalders
Copy link
Contributor

oalders commented Mar 17, 2025

Managing different git remotes can get confusing really quickly. 😄 I hope your job search is going well -- if you're searching.

Copy link
Contributor

@jbarrett jbarrett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pretty succinct and easy to read - nice work! 👍🏻

I wonder if it's worth adding a short paragraph about your experience with the MIDI Rock joystick controller. Time to blow the dust off that controller sitting in your closet, give it a new lease of life with perl superpowers type angle.

@ology
Copy link
Collaborator Author

ology commented Mar 20, 2025

I have zero experience controlling my MIDI Rock.

@ology
Copy link
Collaborator Author

ology commented Mar 20, 2025

FYI: I know how to push to the other PR now. Unfortunately, all my commits are for this PR and I want to keep them all in the same place.

@jbarrett
Copy link
Contributor

I have zero experience controlling my MIDI Rock.

Does that matter? You don't need to go into the ins-and-outs, just that you picked it up again with this in mind. Anyway, just a thought...

@ology
Copy link
Collaborator Author

ology commented Mar 20, 2025

I have zero experience controlling my MIDI Rock.

Does that matter? You don't need to go into the ins-and-outs, just that you picked it up again with this in mind. Anyway, just a thought...

Ok. Fair enough. If it makes the article more exciting, then so be it!

@ology ology requested review from jbarrett and oalders and removed request for jbarrett March 20, 2025 22:16
@ology
Copy link
Collaborator Author

ology commented Mar 21, 2025

@oalders This appears to be ready to roll. :)

@jbarrett
Copy link
Contributor

This looks good from my POV - thanks!

@oalders
Copy link
Contributor

oalders commented Mar 21, 2025

Thanks @ology and @jbarrett! I'm going to target a publish date in the week of March 31, but I will look at this well before then.

@oalders
Copy link
Contributor

oalders commented Mar 23, 2025

@ology the preview is now up to date: https://www-perl-com-pr-427.onrender.com/article/enhancing-your-midi-devices-round-ii/

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.

3 participants