search

boostorg / website-v2-docs

Boost Site Documentation
https://docs.cppalliance.org
Boost Software License 1.0
6 stars 21 forks source link

Add Local Regression Tests topic #221

Closed PeterTurcan closed 3 months ago

PeterTurcan commented 4 months ago

fix #218

Most of the introductory stuff for Regression was already included in the Introduction to testing, and Test Matrix sections. Added a single page on local regression tests.

cppalliance-bot commented 4 months ago

An automated preview of the documentation is available at https://221.site-docs.prtest2.cppalliance.org/contributor-guide/index.html

joaquintides commented 4 months ago

There're some problems with this part:

imagen

So, we need to replace this single link with four links as described above, probably using a layout similar to this:

imagen

(The "Developers" header may be changed to something more pertinent such as "Results"). Now, the location of these four links in the new backend are:

These, again, are absolute links, and boost.io is not the final domain. We can include them in a couple of ways:

  1. As
  2. Via relative URLs redirected to the proper regression.*/xx absolute URLs

@kennethreitz, @sdarwin which option do you think is best? Or maybe there's a better solution I haven't thought of.

Thank you,

PeterTurcan commented 3 months ago

Probably the right approach here is to use the four links that work for now, merge the PR, then file a reasonably high priority issue to change to the four new links when the migration is done and tested?

By doing this the information gets out there and there is no pressure to rush the migration if or when issues arise?

On Sat, Apr 27, 2024 at 2:38 AM joaquintides @.***> wrote:

There're some problems with this part:

imagen.png (view on web) https://github.com/boostorg/website-v2-docs/assets/3283164/1029162a-ed18-440f-930d-d2e5ae68cac8

So, we need to replace this single link with four links as described above, probably using a layout similar to this:

imagen.png (view on web) https://github.com/boostorg/website-v2-docs/assets/3283164/ed6419e1-5881-4748-9cb0-5abb5e39aa77

(The "Developers" header may be changed to something more pertinent such as "Results"). Now, the location of these four links in the new backend are:

These, again, are absolute links, and boost.io is not the final domain. We can include them in a couple of ways:

  1. As

  2. Via relative URLs redirected to the proper regression.*/xx absolute URLs

@kennethreitz https://github.com/kennethreitz, @sdarwin https://github.com/sdarwin which option do you think is best? Or maybe there's a better solution I haven't thought of.

Thank you,

— Reply to this email directly, view it on GitHub https://github.com/boostorg/website-v2-docs/pull/221#issuecomment-2080432592, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEIBS2JVIXS3TWW6WRCCLJLY7NWZVAVCNFSM6AAAAABGXQNRNGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBQGQZTENJZGI . You are receiving this because you authored the thread.Message ID: @.***>

joaquintides commented 3 months ago

Hi Peter, yes, let's do as you propose. Other solutions are probably too complicated for what's esentially a one-time event (the migration).

PeterTurcan commented 3 months ago

@joaquintides OK - done!

cppalliance-bot commented 3 months ago

An automated preview of the documentation is available at https://221.site-docs.prtest2.cppalliance.org/contributor-guide/index.html

joaquintides commented 3 months ago

Merged, thank you!