Skip to content

Conversation

@manikanta-hitunik-com
Copy link

316
317

316
317
@mikeckennedy
Copy link
Member

Thanks a bunch @manikanta-hitunik-com But can you please do something different in the future.

It looks like you take the mp3 and then directly generate the transcript even though we already have the files here in this repo. For example, for this PR, it's:

  • 316-flask-2-0.txt
  • 317-python-flask-at-fec.txt

Then when you add 316.txt entirely separately rather than updating 316-flask-2-0.txt it's hard for me to know which one is the correct one to use on the site (which is newer) and also hard to see what changes have been made.

So in the future, please look here to grab the actual txt files. Then make changes to them and commit those changes back in a PR like this.

Sound good?

@mikeckennedy mikeckennedy merged commit c002276 into talkpython:master May 27, 2021
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.

2 participants