Skip to content

2.x#1617

Closed
jarednova wants to merge 2273 commits into1.xfrom
2.x
Closed

2.x#1617
jarednova wants to merge 2273 commits into1.xfrom
2.x

Conversation

@jarednova
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown

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
Copy Markdown

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
Copy Markdown

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
Copy Markdown

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
Copy Markdown

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
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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.