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

2.x #1617

Closed
wants to merge 2,273 commits into from
Closed

2.x #1617

wants to merge 2,273 commits into from

Conversation

jarednova
Copy link
Member

Ticket: #1580

This will track the progress and tests for 2.x into master

jarednova added a commit that referenced this pull request Dec 29, 2017
jarednova added a commit that referenced this pull request Dec 29, 2017
jarednova added a commit that referenced this pull request Dec 29, 2017
@timber timber deleted a comment from coveralls Dec 29, 2017
@jarednova jarednova added the 2.0 label Dec 29, 2017
@timber timber deleted a comment from coveralls Jan 3, 2018
@timber timber deleted a comment from coveralls Jan 3, 2018
@timber timber deleted a comment from coveralls Jan 3, 2018
@timber timber deleted a comment from coveralls Jan 5, 2018
@timber timber deleted a comment from coveralls Jan 5, 2018
@timber timber deleted a comment from coveralls Jan 5, 2018
@timber timber deleted a comment from coveralls Jan 5, 2018
@timber timber deleted a comment from coveralls Jan 15, 2018
@timber timber deleted a comment from coveralls Jan 15, 2018
@timber timber deleted a comment from coveralls Jan 15, 2018
@timber timber deleted a comment from coveralls Jan 15, 2018
jarednova added a commit that referenced this pull request Mar 5, 2018

// Menu item with _menu_item_target set to ''
$item = $items[2];
$this->assertFalse( '_self', $item->target() );
Copy link
Member Author

Choose a reason for hiding this comment

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

@gchtr wanted to check on this test (it's failing right now). Should $item->target() register as _self or false in this scenario?

Copy link
Member

Choose a reason for hiding this comment

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

@jarednova Oh! I don’t know why it wasn’t failing for me. But that should probably be:

$this->assertEquals( '_self', $item->target() );

It should definitely return _self. I copied the assertFalse from the test above and forgot to change the assertion method.

Copy link
Member Author

Choose a reason for hiding this comment

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

Cool, that's what I thought! I have a patch I'll pull in to 2.x

Copy link
Member

Choose a reason for hiding this comment

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

jarednova added a commit that referenced this pull request May 30, 2018
jarednova added a commit that referenced this pull request Jun 1, 2018
jarednova added a commit that referenced this pull request Jun 1, 2018
@jarednova jarednova mentioned this pull request Jun 1, 2018
@codecov
Copy link

codecov bot commented Sep 19, 2018

Codecov Report

Merging #1617 into master will decrease coverage by 0.52%.
The diff coverage is 90.15%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1617      +/-   ##
============================================
- Coverage     94.83%   94.31%   -0.53%     
+ Complexity     1527     1507      -20     
============================================
  Files            48       48              
  Lines          3585     3799     +214     
============================================
+ Hits           3400     3583     +183     
- Misses          185      216      +31
Impacted Files Coverage Δ Complexity Δ
lib/Cache/WPObjectCacheAdapter.php 42.85% <ø> (ø) 3 <0> (ø) ⬇️
lib/PostPreview.php 96.29% <ø> (ø) 38 <0> (ø) ⬇️
lib/FunctionWrapper.php 92% <ø> (+1.09%) 14 <0> (-3) ⬇️
lib/PostCollection.php 100% <ø> (ø) 18 <0> (ø) ⬇️
lib/PostQuery.php 100% <ø> (ø) 5 <0> (ø) ⬇️
lib/Request.php 100% <ø> (ø) 2 <0> (ø) ⬇️
lib/Twig_Function.php 0% <ø> (ø) 0 <0> (ø) ⬇️
lib/Integrations.php 91.66% <ø> (ø) 6 <0> (ø) ⬇️
lib/PostsIterator.php 100% <ø> (ø) 1 <0> (ø) ⬇️
lib/Pagination.php 100% <ø> (ø) 42 <0> (ø) ⬇️
... and 32 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bf826bd...7167425. Read the comment docs.

@codecov
Copy link

codecov bot commented Sep 19, 2018

Codecov Report

Merging #1617 into master will decrease coverage by 0.52%.
The diff coverage is 90.15%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1617      +/-   ##
============================================
- Coverage     94.83%   94.31%   -0.53%     
+ Complexity     1527     1507      -20     
============================================
  Files            48       48              
  Lines          3585     3799     +214     
============================================
+ Hits           3400     3583     +183     
- Misses          185      216      +31
Impacted Files Coverage Δ Complexity Δ
lib/Cache/WPObjectCacheAdapter.php 42.85% <ø> (ø) 3 <0> (ø) ⬇️
lib/PostPreview.php 96.29% <ø> (ø) 38 <0> (ø) ⬇️
lib/FunctionWrapper.php 92% <ø> (+1.09%) 14 <0> (-3) ⬇️
lib/PostCollection.php 100% <ø> (ø) 18 <0> (ø) ⬇️
lib/PostQuery.php 100% <ø> (ø) 5 <0> (ø) ⬇️
lib/Request.php 100% <ø> (ø) 2 <0> (ø) ⬇️
lib/Twig_Function.php 0% <ø> (ø) 0 <0> (ø) ⬇️
lib/Integrations.php 91.66% <ø> (ø) 6 <0> (ø) ⬇️
lib/PostsIterator.php 100% <ø> (ø) 1 <0> (ø) ⬇️
lib/Pagination.php 100% <ø> (ø) 42 <0> (ø) ⬇️
... and 32 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fae5977...80ee371. Read the comment docs.

4 similar comments
@codecov
Copy link

codecov bot commented Sep 19, 2018

Codecov Report

Merging #1617 into master will decrease coverage by 0.52%.
The diff coverage is 90.15%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1617      +/-   ##
============================================
- Coverage     94.83%   94.31%   -0.53%     
+ Complexity     1527     1507      -20     
============================================
  Files            48       48              
  Lines          3585     3799     +214     
============================================
+ Hits           3400     3583     +183     
- Misses          185      216      +31
Impacted Files Coverage Δ Complexity Δ
lib/Cache/WPObjectCacheAdapter.php 42.85% <ø> (ø) 3 <0> (ø) ⬇️
lib/PostPreview.php 96.29% <ø> (ø) 38 <0> (ø) ⬇️
lib/FunctionWrapper.php 92% <ø> (+1.09%) 14 <0> (-3) ⬇️
lib/PostCollection.php 100% <ø> (ø) 18 <0> (ø) ⬇️
lib/PostQuery.php 100% <ø> (ø) 5 <0> (ø) ⬇️
lib/Request.php 100% <ø> (ø) 2 <0> (ø) ⬇️
lib/Twig_Function.php 0% <ø> (ø) 0 <0> (ø) ⬇️
lib/Integrations.php 91.66% <ø> (ø) 6 <0> (ø) ⬇️
lib/PostsIterator.php 100% <ø> (ø) 1 <0> (ø) ⬇️
lib/Pagination.php 100% <ø> (ø) 42 <0> (ø) ⬇️
... and 32 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fae5977...80ee371. Read the comment docs.

@codecov
Copy link

codecov bot commented Sep 19, 2018

Codecov Report

Merging #1617 into master will decrease coverage by 0.52%.
The diff coverage is 90.15%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1617      +/-   ##
============================================
- Coverage     94.83%   94.31%   -0.53%     
+ Complexity     1527     1507      -20     
============================================
  Files            48       48              
  Lines          3585     3799     +214     
============================================
+ Hits           3400     3583     +183     
- Misses          185      216      +31
Impacted Files Coverage Δ Complexity Δ
lib/Cache/WPObjectCacheAdapter.php 42.85% <ø> (ø) 3 <0> (ø) ⬇️
lib/PostPreview.php 96.29% <ø> (ø) 38 <0> (ø) ⬇️
lib/FunctionWrapper.php 92% <ø> (+1.09%) 14 <0> (-3) ⬇️
lib/PostCollection.php 100% <ø> (ø) 18 <0> (ø) ⬇️
lib/PostQuery.php 100% <ø> (ø) 5 <0> (ø) ⬇️
lib/Request.php 100% <ø> (ø) 2 <0> (ø) ⬇️
lib/Twig_Function.php 0% <ø> (ø) 0 <0> (ø) ⬇️
lib/Integrations.php 91.66% <ø> (ø) 6 <0> (ø) ⬇️
lib/PostsIterator.php 100% <ø> (ø) 1 <0> (ø) ⬇️
lib/Pagination.php 100% <ø> (ø) 42 <0> (ø) ⬇️
... and 32 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fae5977...80ee371. Read the comment docs.

@codecov
Copy link

codecov bot commented Sep 19, 2018

Codecov Report

Merging #1617 into master will decrease coverage by 0.52%.
The diff coverage is 90.15%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1617      +/-   ##
============================================
- Coverage     94.83%   94.31%   -0.53%     
+ Complexity     1527     1507      -20     
============================================
  Files            48       48              
  Lines          3585     3799     +214     
============================================
+ Hits           3400     3583     +183     
- Misses          185      216      +31
Impacted Files Coverage Δ Complexity Δ
lib/Cache/WPObjectCacheAdapter.php 42.85% <ø> (ø) 3 <0> (ø) ⬇️
lib/PostPreview.php 96.29% <ø> (ø) 38 <0> (ø) ⬇️
lib/FunctionWrapper.php 92% <ø> (+1.09%) 14 <0> (-3) ⬇️
lib/PostCollection.php 100% <ø> (ø) 18 <0> (ø) ⬇️
lib/PostQuery.php 100% <ø> (ø) 5 <0> (ø) ⬇️
lib/Request.php 100% <ø> (ø) 2 <0> (ø) ⬇️
lib/Twig_Function.php 0% <ø> (ø) 0 <0> (ø) ⬇️
lib/Integrations.php 91.66% <ø> (ø) 6 <0> (ø) ⬇️
lib/PostsIterator.php 100% <ø> (ø) 1 <0> (ø) ⬇️
lib/Pagination.php 100% <ø> (ø) 42 <0> (ø) ⬇️
... and 32 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fae5977...80ee371. Read the comment docs.

@codecov
Copy link

codecov bot commented Sep 19, 2018

Codecov Report

Merging #1617 into master will decrease coverage by 0.52%.
The diff coverage is 90.15%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1617      +/-   ##
============================================
- Coverage     94.83%   94.31%   -0.53%     
+ Complexity     1527     1507      -20     
============================================
  Files            48       48              
  Lines          3585     3799     +214     
============================================
+ Hits           3400     3583     +183     
- Misses          185      216      +31
Impacted Files Coverage Δ Complexity Δ
lib/Cache/WPObjectCacheAdapter.php 42.85% <ø> (ø) 3 <0> (ø) ⬇️
lib/PostPreview.php 96.29% <ø> (ø) 38 <0> (ø) ⬇️
lib/FunctionWrapper.php 92% <ø> (+1.09%) 14 <0> (-3) ⬇️
lib/PostCollection.php 100% <ø> (ø) 18 <0> (ø) ⬇️
lib/PostQuery.php 100% <ø> (ø) 5 <0> (ø) ⬇️
lib/Request.php 100% <ø> (ø) 2 <0> (ø) ⬇️
lib/Twig_Function.php 0% <ø> (ø) 0 <0> (ø) ⬇️
lib/Integrations.php 91.66% <ø> (ø) 6 <0> (ø) ⬇️
lib/PostsIterator.php 100% <ø> (ø) 1 <0> (ø) ⬇️
lib/Pagination.php 100% <ø> (ø) 42 <0> (ø) ⬇️
... and 32 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fae5977...80ee371. Read the comment docs.

@timber timber deleted a comment from codecov bot Nov 27, 2018
Levdbas and others added 27 commits October 24, 2023 20:39
Co-authored-by: Erik van der Bas <erik@basedonline.nl>
# Conflicts:
#	bin/timber.php
#	lib/Timber.php
#	readme.txt
@gchtr
Copy link
Member

gchtr commented Nov 9, 2023

Timber 2.0 is released: https://github.com/timber/timber/releases/tag/2.0.0.

We kept the 2.x branch and made it the default branch. We’re not going to merge this into master (which was renamed to 1.x). So we can close this pull request.

@gchtr gchtr closed this Nov 9, 2023
@szepeviktor
Copy link
Contributor

One major version bump of Timber equals 5 children from a female 👱🏻‍♀️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet