Sitelet https://github.com/TheRenegadeCoder/sample-programs-website/pull/375
Skip to content

Added Baklava in Kotlin Article - #375

Merged
jrg94 merged 12 commits into
TheRenegadeCoder:masterfrom
stuin:master
Oct 26, 2020
Merged

jrg94 merged 12 commits into
TheRenegadeCoder:masterfrom
stuin:master

Conversation

@stuin

@stuin stuin commented Oct 23, 2020 •

Copy link
Copy Markdown
Member

Thanks for helping us build up our knowledge base!
What kind of pull request hav you brought us today?

Pull Request Type

  • Sample Program Documentation
  • Language Documentation
  • Project Documentation
  • Other (explain below)

Notes

Feel free to share any details you feel are relevant.

@stuin

stuin commented Oct 23, 2020 •

Copy link
Copy Markdown
Member Author

Fixed #192

@jrg94
jrg94 requested a review from a team October 23, 2020 20:58
@jrg94 jrg94 self-assigned this Oct 23, 2020
@jrg94 jrg94 added the sample program Sample program documentation label Oct 23, 2020

@jrg94 jrg94 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks great! Any chance you'd want to help with the featured images? It's pretty easy to do. I usually just pick something from here: https://pixabay.com/

Then, you can drop the image in the assets/sources/ folder, and the CI will generate the titled image for you. (Don't forget to put the file name in the header here.)

@stuin

stuin commented Oct 24, 2020

Copy link
Copy Markdown
Member Author

Looks like it needs write access to generate all the images?

@jrg94

jrg94 commented Oct 24, 2020 •

Copy link
Copy Markdown
Member

Interesting! Is it because you added the files? Let me see if I can set this up to always push as me.

@jrg94

jrg94 commented Oct 24, 2020

Copy link
Copy Markdown
Member

I'm going to try rerunning it. If that doesn't work, I'll add you to core. If that doesn't work, I'll try committing as myself.

@jrg94

jrg94 commented Oct 24, 2020 •

Copy link
Copy Markdown
Member

Actually, this shouldn't be an issue since it commits to the current branch... hmmmm! I thought maybe the issue was pushing to master which is protected. Maybe because the changes are on a fork?

@jrg94

jrg94 commented Oct 24, 2020 •

Copy link
Copy Markdown
Member

Can you try pulling from master?

I'm seeing this:

> Unable to get commit from workflow event: trying with the GitHub API...

Not sure what this means.

@stuin

stuin commented Oct 24, 2020

Copy link
Copy Markdown
Member Author

Error: Error: Pushing to https://github.com/TheRenegadeCoder/sample-programs-website
remote: Permission to TheRenegadeCoder/sample-programs-website.git denied to github-actions[bot].

@jrg94

jrg94 commented Oct 24, 2020 •

Copy link
Copy Markdown
Member

I saw that error, but the error I posted happens even before that. I think it's possibly a fork issue. I'll investigate a bit more. Author appears to be null. Let me force it as me.

@jrg94

jrg94 commented Oct 24, 2020

Copy link
Copy Markdown
Member

Can you try pulling from master once more?

@jrg94

jrg94 commented Oct 24, 2020

Copy link
Copy Markdown
Member

I really think this has something to do with you being on a fork. It looks like it's trying to push to Master on this repo despite the change being on your repo. That's why it can't find a commit to work from. I'll sent a bug report out to the author of the tool. Perhaps they can shed some light on it.

@jrg94

jrg94 commented Oct 26, 2020

Copy link
Copy Markdown
Member

I'm going to merge this. The website build will likely fail, so I will make a separate pull request with an additional source image.

@jrg94
jrg94 merged commit 82a83f8 into TheRenegadeCoder:master Oct 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sample program Sample program documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants