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

1931 Canadian Census #4875

Merged
merged 21 commits into from
Sep 21, 2023
Merged

1931 Canadian Census #4875

merged 21 commits into from
Sep 21, 2023

Conversation

manf0001
Copy link
Contributor

The Canadian 1931 Census is digitized and starting to be seen online at Ancestry or the Canadian Archives for download. Adding that year to webtrees.

@fisharebest
Copy link
Owner

Could you also add/update the corresponding test scripts?

  • app/Census/CensusOfCanadaTest.php
  • app/Census/CensusOfCanadaTest1931.php

They should be pretty straightforward. Just look at one of the other test scripts.

Thanks!

@manf0001
Copy link
Contributor Author

sorry, I didn't know about the test scripts.. will work on that, then uploaded and issue a PR for it ASAP

@codecov
Copy link

codecov bot commented Sep 20, 2023

Codecov Report

Merging #4875 (8b41848) into main (a7b47bd) will increase coverage by 0.08%.
Report is 7 commits behind head on main.
The diff coverage is 100.00%.

❗ Current head 8b41848 differs from pull request most recent head 724c6dc. Consider uploading reports for the commit 724c6dc to get more accurate results

@@             Coverage Diff              @@
##               main    #4875      +/-   ##
============================================
+ Coverage     30.97%   31.06%   +0.08%     
- Complexity    11377    11379       +2     
============================================
  Files          1169     1170       +1     
  Lines         47887    47929      +42     
============================================
+ Hits          14835    14890      +55     
+ Misses        33052    33039      -13     
Files Changed Coverage Δ
app/Census/CensusColumnNationality.php 100.00% <ø> (ø)
app/Census/CensusOfCanada.php 100.00% <100.00%> (+100.00%) ⬆️
app/Census/CensusOfCanada1931.php 100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

@fisharebest fisharebest merged commit f71a458 into fisharebest:main Sep 21, 2023
10 checks passed
@fisharebest
Copy link
Owner

Thanks! This will be included in the next release.

fisharebest added a commit that referenced this pull request Sep 21, 2023
* Update CensusOfCanada.php

Adding 1931

* Create CensusOfCanada1931.php

1931 Canadian Census

* Update CensusOfCanada1931.php

* Update CensusOfCanada1931.php

* Create CensusOfCanadaTest1931.php

* Delete tests/app/Census/CensusOfCanadaTest1931.php

* Create CensusOfCanada1931Test.php

* Update CensusOfCanada1931.php

Added some missing lines and re-worded the description for the last couple of columns

* Update CensusOfCanada1931.php

removed brackets from two descriptions to conform with rest of the column descriptions

* Update CensusOfCanada1931Test.php

* Update CensusOfCanada1931Test.php

Fix spacing

* Update CensusOfCanadaTest.php

Added info for Census of Canada Test

* Update CensusColumnNationality.php

Added two more Nationalities for Census

* Update CensusOfCanada1931Test.php

removing tabs and replaced with spaces

* Update CensusOfCanadaTest.php

forgot to add line for 1891 census

* Update CensusOfCanadaTest.php

change language

* Update CensusOfCanada1931.php

Fixed Typo

* Update CensusOfCanada1931Test.php

added missing .

* Update CensusOfCanada1931Test.php

* Update CensusOfCanada1931.php

* Update CensusColumnNationality.php

---------

Co-authored-by: Greg Roach <[email protected]>
ddrury pushed a commit to ddrury/webtrees that referenced this pull request Nov 27, 2024
* Update CensusOfCanada.php

Adding 1931

* Create CensusOfCanada1931.php

1931 Canadian Census

* Update CensusOfCanada1931.php

* Update CensusOfCanada1931.php

* Create CensusOfCanadaTest1931.php

* Delete tests/app/Census/CensusOfCanadaTest1931.php

* Create CensusOfCanada1931Test.php

* Update CensusOfCanada1931.php

Added some missing lines and re-worded the description for the last couple of columns

* Update CensusOfCanada1931.php

removed brackets from two descriptions to conform with rest of the column descriptions

* Update CensusOfCanada1931Test.php

* Update CensusOfCanada1931Test.php

Fix spacing

* Update CensusOfCanadaTest.php

Added info for Census of Canada Test

* Update CensusColumnNationality.php

Added two more Nationalities for Census

* Update CensusOfCanada1931Test.php

removing tabs and replaced with spaces

* Update CensusOfCanadaTest.php

forgot to add line for 1891 census

* Update CensusOfCanadaTest.php

change language

* Update CensusOfCanada1931.php

Fixed Typo

* Update CensusOfCanada1931Test.php

added missing .

* Update CensusOfCanada1931Test.php

* Update CensusOfCanada1931.php

* Update CensusColumnNationality.php

---------

Co-authored-by: Greg Roach <[email protected]>
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