Making WordPress.org

Changeset 11238


Ignore:
Timestamp:
09/15/2021 07:15:56 AM (3 years ago)
Author:
dd32
Message:

Plugin Directory: When release confirmations are enabled, always build the trunk plugin-name.zip zip.

When release confirmations are enabled, trunk can never be set as the stable tag, so it's always safe to build the trunk zip.

There's still a limitation here - If a commit introduces a new stable release AND modifies trunk in the same commit, the trunk ZIP won't be updated until the release is confirmed.
It'd be nice to still rebuild the ZIP in that case, so a follow up change may still be required.

See #5352.
See #5901.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/cli/class-import.php

    r10971 r11238  
    314314     * @param string $plugin_slug            The plugin slug.
    315315     * @param string $stable_tag             The new stable tag.
    316      * @param string $current_stable_tag     The new stable tag.
     316     * @param string $current_stable_tag     The current stable tag.
    317317     * @param array  $svn_changed_tags       The list of SVN tags modified since last import.
    318318     * @param string $svn_revision_triggered The SVN revision which triggered the rebuild.
     
    333333        if ( $plugin->release_confirmation ) {
    334334            foreach ( $versions_to_build as $i => $tag ) {
     335                // Trunk should always be built, and will never be set as the stable tag when confirmations are enabled.
     336                if ( 'trunk' === $tag ) {
     337                    continue;
     338                }
     339
    335340                $release = Plugin_Directory::get_release( $plugin, $tag );
    336341
Note: See TracChangeset for help on using the changeset viewer.