No Code Attached Yet
avatar astridx
astridx
12 Feb 2023

Steps to reproduce the issue

Clone the branch 4.3-dev of this repo and open the file .gitignore.

Expected result

.gitignore should look like in #39721

Actual result

The text output is not there. See

/tests/cypress/screenshots

Although #39721 has been merged, it does not appear in the historie of the file .gitignore.

avatar astridx astridx - open - 12 Feb 2023
avatar joomla-cms-bot joomla-cms-bot - change - 12 Feb 2023
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 12 Feb 2023
avatar brianteeman
brianteeman - comment - 12 Feb 2023

Your pull request #39722 was merged after #39721 and reverted the change

avatar wojtekxtx
wojtekxtx - comment - 13 Feb 2023

@brianteeman why was this change reverted?

avatar HLeithner
HLeithner - comment - 18 Feb 2023

@astridx as you reverted it your self you have to decide which is the right one ;-) looking into the config file we use the output path

  videosFolder: 'tests/cypress/output/videos',
  screenshotsFolder: 'tests/cypress/output/screenshots',

Can you please fix it again or did @laoneo see a valid reason why the .gitígnore has been changed in #39722 ?

avatar fancyFranci
fancyFranci - comment - 18 Feb 2023

I assume wrong merge conflict resolving @wojtekxtx.
@astridx can you create a new PR to bring back the correct configs please?

avatar wojtekxtx
wojtekxtx - comment - 18 Feb 2023

@fancyFranci Thanks for reply ?

Instead of creating new PR and messing up history, would not be easier to just git reset --hard to specific commit? Just thinking........

avatar laoneo
laoneo - comment - 18 Feb 2023

That won't work as all the clones are messed up then.

avatar astridx astridx - change - 19 Feb 2023
Status New Closed
Closed_Date 0000-00-00 00:00:00 2023-02-19 11:18:23
Closed_By astridx
avatar astridx
astridx - comment - 19 Feb 2023

See #39890

avatar astridx astridx - close - 19 Feb 2023

Add a Comment

Login with GitHub to post a comment