Skip to content
This repository has been archived by the owner on Dec 1, 2019. It is now read-only.

NEW ISSUES SHOULD BE CREATED ON TRAC #991

Open
pattonwebz opened this issue Nov 10, 2019 · 3 comments
Open

NEW ISSUES SHOULD BE CREATED ON TRAC #991

pattonwebz opened this issue Nov 10, 2019 · 3 comments

Comments

@pattonwebz
Copy link
Member

pattonwebz commented Nov 10, 2019

From this point on any new issues should be opened on the core trac instead of at GitHub to save them needing to be transitioned over there manually at a later point.

Please prefix tickets with Twenty Twenty: so that they are easy to find and collect with a search or filters. Thank you!

https://core.trac.wordpress.org/search?q=twenty+twenty%3A

@pattonwebz pattonwebz pinned this issue Nov 10, 2019
nielslange added a commit that referenced this issue Nov 11, 2019
@ pattonwebz: I created this issue template, in case someone missed your issue #991.
pattonwebz pushed a commit that referenced this issue Nov 11, 2019
* Update issue templates

@ pattonwebz: I created this issue template, in case someone missed your issue #991.

* Update .github/ISSUE_TEMPLATE/create-new-issues-on-trac.md

Co-Authored-By: Aurorum <torres126@clubpenguinmountains.com>

* Update create-new-issues-on-trac.md
@ottok
Copy link

ottok commented Nov 14, 2019

Why do you link to trac main page here? Please link to the list of twentytwenty issues directly. Thanks!

The commit 1025b52#diff-3d9639f99b2cd6bda90b892bea090bef broke this plugin for PHP composer users, as there is a dotfile now shipped with the plugin and composer does not handle well those cases. This is a similar bug as when akismet started shipping a .htaccess file in May 2017.

# composer install
Loading composer repositories with package information
Installing dependencies (including require-dev) from lock file
Package operations: 4 installs, 0 updates, 0 removals
  - Installing johnpbloch/wordpress-core (5.3.0): Loading from cache
  - Installing wpackagist-plugin/auto-post-thumbnail (3.6.0): Downloading (100%)         
  - Installing wpackagist-plugin/google-analytics-dashboard-for-wp (5.3.9): Downloading (100%)         
  - Installing wpackagist-theme/twentynineteen (1.4): Downloading (100%)         
Generating autoload files
> WordPress\Installer::symlinkWPContent
Script WordPress\Installer::symlinkWPContent handling the post-install-cmd event terminated with an exception

                                                                                               
  [ErrorException]                                                                             
  rmdir(/data/wordpress/htdocs/wordpress/wp-content/themes/twentytwenty): Directory not empty                                                                                            

@pattonwebz
Copy link
Member Author

Hey @ottok, we can certainly update the issue template so it links directly to the tickets, do you think also linking directly to the new ticket page would be good too?

As for composer problems with dotfiles in the directory - that is unfortunate and I will look into that and see what can be done, thank you for reporting.

@ottok
Copy link

ottok commented Nov 14, 2019

No, just link to the issue overview. You want users to skim existing issues before going to file new ones.

For the composer issue I wrote a local fix to our copy of the general WordPress/Install.php used in in many gitified WordPress templates: Seravo/wordpress@a42412c

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

No branches or pull requests

2 participants