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.

Sign up for GitHub

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

Jump to bottom

chore:add encode #40

Closed
wants to merge 54 commits into from
Closed

chore:add encode #40

wants to merge 54 commits into from

Conversation

guizaida
Copy link

Types of changes

  • Bugfix
  • New feature
  • Refactoring
  • Breaking change (any change that would cause existing functionality to not work as expected)
  • Documentation Update
  • Other (please describe)

Description

Checklist:

  • Add test cases to all the changes you introduce
  • Run inv style locally to ensure all linter checks pass
  • Run inv test locally to ensure all test cases pass
  • Run inv secure locally to ensure no major vulnerability is introduced
  • Update the documentation if necessary

Steps to Test This Pull Request

Expected behavior

Related Issue

Additional context

josix and others added 30 commits August 2, 2021 20:30
@josix
Update archive-pycontw.yml
c8d81c3
@josix
fix: Resolve wrong base url issue
aae3868
@josix
Merge pull request pycontw#11 from josix/main
38700f8 
fix: Resolve wrong base url issue
@mozixreality
local switch
3ca8b4c
@mozixreality
base_url -> base url
2de3448
@mozixreality
Update README.md
cade367
@josix
Merge pull request pycontw#13 from pycontw/mozix
0e2b9a8 
Mozix
@allen91wu
fix(main.py): crawler all page in specific year
98e5588
@allen91wu
fix(main.py): crawler sponsorship prospectus page
0e4d1a4
@allen91wu
fix(main.py): remove nav-lang-form method
a9cb450
@allen91wu
fix(main.py): locale switch
e8d333a
@allen91wu
fix(main.py): fix css-background url
ca927fe
@allen91wu
style(main.py): fix coding style
a04ebce
@josix
Merge pull request pycontw#17 from allen91wu/fix-bug
539f3ee 
Fix locale-switch and bg-image crawling issue
@josix
Update archive-pycontw.yml
1793860
@josix
fix: Fix 2020/zh-hant/sponsor/sponsor/ page 404 error
c5f1fbb
@josix
feat: Update to scrape favicon
0175d2e
@josix
feat: Remove redundant code in get_assets
ce0cc14
@josix
Merge pull request pycontw#18 from josix/main
d713916 
feat: Update to scrape favicon
@allen91wu
fix(website_2020): fix crawler of warmup event page
9d3d49a
@allen91wu
style(black): reformat coding style via black
559ef46
@allen91wu
Merge pull request pycontw#19 from allen91wu/fix-crawler
50d281c 
Fix crawler of 2020 warmup event page
@josix
ci: Add Year action input and validation
54cb81d
@josix
ci: Update to use python3.9 and fix wrong working dir
5608e7e
@josix
ci: Avoid clean branch before deploy
5271427
@josix
Merge pull request pycontw#20 from josix/update/github_crawling_action
54487c9 
Update/GitHub crawling action
@josix
build: Update to pin used packages
b7aa412
@mattwang44
chore: rm unused Dockerfile & compose yaml files
16073c5
@mattwang44
doc: update README for repo description and script usage
fd305df
@josix
Merge pull request pycontw#23 from josix/main
a75e355 
build: Update to pin used packages
josix and others added 24 commits October 25, 2021 00:02
@josix
Delete .DS_Store
b52b3e5
@josix
Merge pull request pycontw#24 from pycontw/rm-docker
e54f8b5 
Remove unused Dockerfile & compose yaml files and update README
@josix
fix: Unify path format by inoking urlparser and prevent keep old base…
e4a7317 
…_url
@josix
chore: update pipenv serve command name
b6dba3c
@josix
docs: Fix worng introduction for running local server
12d635d
@josix
Merge pull request pycontw#26 from josix/fix/compability-with-sites-n…
d1ebba5 
…ot-start-with-year

Fix compatibility with deployed static sites
@allen91wu
refactor(cruft): add linter and formater via cruft
e049fbe
@allen91wu
refactor(cruft): add project description
c04865b
@allen91wu
Merge pull request pycontw#28 from pycontw/add-linter-and-formater
cbc4383 
Add linter and formatter
@allen91wu
ci(main-updated.yaml): remove publish-github-page action
9cc9e1a
@allen91wu
Merge pull request pycontw#29 from pycontw/fix-workflow
325807e 
ci(main-updated.yaml): remove publish-github-page action
@allen91wu
ci(main-updated.yaml): change access token
0b2ec83
@josix
Merge pull request pycontw#30 from pycontw/fix-token
f1e7849 
ci(main-updated.yaml): change access token
@github-actions
bump: version 0.0.1 → 0.1.0
6e506ff
@Darkborderman
refactor: Add basic common strucutre
b6756de
@Darkborderman
refactor: Refine dataio functions
7809aae
@Darkborderman
refactor: Add crawler website base structure
11c277e
@Darkborderman
refactor: Refine common modules
b90b544
@Darkborderman
refactor: Refactor crawlers
c6c2eeb
@Darkborderman
refactor: Update main function
3444bcc
@Darkborderman
refactor: Refactor crawler classes
653e3d9
@josix
Merge pull request pycontw#36 from Darkborderman/refactor/modulize
824cc09 
Refactor crawler function according to issue pycontw#16
@github-actions
bump: version 0.1.0 → 0.1.1
c3592da
@guizaida
chore:add encode
69e555a
@josix josix changed the base branch from main to gh-pages May 15, 2022 16:38
@josix josix self-requested a review May 15, 2022 16:39
josix
josix requested changes May 16, 2022
View reviewed changes
Copy link
Contributor

@josix josix left a comment

Choose a reason for hiding this comment

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

Oops. It seems like the code pushed to pycontw/pycontw-2014apac-archive#2 is the correct one. Could you help me force-push commits on this pull request? Thanks!

@josix josix closed this Jun 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@josix josix josix requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@guizaida @josix @mozixreality @allen91wu @mattwang44 @Darkborderman