!zXiOpjSkFTvtMpsenJ:gitter.im

tldr-pages/tldr

384 Members
📚 Collaborative cheatsheets for console commands. Repository: https://github.com/tldr-pages/tldr Website: https://tldr.sh Offtopic room: https://matrix.to/#/#tldr-pages_off-topic:gitter.im Space: https://matrix.to/#/#tldr-pages-project:matrix.org14 Servers

Load older messages


SenderMessageTime
27 Feb 2024
@kbdk:matrix.orgK.B.Dharun KrishnaSame, I too can't figure out what the issue is probably a rogue space. Will check it locally after a few minutes.15:09:44
@kbdk:matrix.orgK.B.Dharun Krishna
In reply to @acuteenvy:matrix.org

I tested all other official clients:

  • the Python client fails one test - it doesn't support macos as an alias for osx,
  • the C client fails all language tests (because it doesn't support translations) + it doesn't convert uppercase letters to lowercase,
  • the Rust client passes.
Regarding this, since we don't require clients to implement language flags do we need to have it in main compliance testing? Seth: or is it part of optional checks?
15:27:13
@kbdk:matrix.orgK.B.Dharun KrishnaFixed it, the issue was a hyphen being written as an underscore.15:41:25
@sethi:one.ems.hostSeth
In reply to @kbdk:matrix.org
Regarding this, since we don't require clients to implement language flags do we need to have it in main compliance testing? Seth: or is it part of optional checks?

My interprettation is that the language flag is optional, but respecting the language envvars is mandatory.

For example, a client doesn't have to accept -L nl, but if the envcar LANG=nl is set, it must be respected unless specified otherwise.

15:44:05
@sethi:one.ems.hostSeth
In reply to @kbdk:matrix.org
Regarding this, since we don't require clients to implement language flags do we need to have it in main compliance testing? Seth: or is it part of optional checks?
*

My interprettation is that the language flag is optional, but respecting the language envvars is mandatory.

For example, a client doesn't have to accept -L nl, but if the envvar LANG=nl is set, it must be respected unless specified otherwise.

15:44:31
@sethi:one.ems.hostSeth
In reply to @kbdk:matrix.org
Regarding this, since we don't require clients to implement language flags do we need to have it in main compliance testing? Seth: or is it part of optional checks?
*

My interprettation is that the language flag is optional, but respecting the language envvars is mandatory.

For example, a client doesn't have to accept -L nl, but if the envvar LANG=nl is set, it must be respected unless specified otherwise.

If a client has access to environment variables, it MUST use them to derive the preferred user language as described in the next paragraphs.

— https://github.com/tldr-pages/tldr/blob/main/CLIENT-SPECIFICATION.md#language

15:45:05
@sethi:one.ems.hostSeth
In reply to @kbdk:matrix.org
Regarding this, since we don't require clients to implement language flags do we need to have it in main compliance testing? Seth: or is it part of optional checks?
*

My interprettation is that the language flag is optional, but respecting the language envvars is mandatory.

For example, a client doesn't have to accept -L nl, but if the envvar LANG=nl is set, it must be respected unless specified otherwise.

If a client has access to environment variables, it MUST use them to derive the preferred user language as described in the next paragraphs.
…
Clients SHOULD offer options to configure or override the language using configuration files or even command-line options

— https://github.com/tldr-pages/tldr/blob/main/CLIENT-SPECIFICATION.md#language

15:46:01
@sethi:one.ems.hostSeth
In reply to @sethi:one.ems.host

My interprettation is that the language flag is optional, but respecting the language envvars is mandatory.

For example, a client doesn't have to accept -L nl, but if the envcar LANG=nl is set, it must be respected unless specified otherwise.

If it's alright with others, I can transfer it to the tldr-pages repo now. Seems like the right time now that I'm more openly prompting discussion.
15:50:02
@sethi:one.ems.hostSeth
In reply to @sethi:one.ems.host

My interprettation is that the language flag is optional, but respecting the language envvars is mandatory.

For example, a client doesn't have to accept -L nl, but if the envcar LANG=nl is set, it must be respected unless specified otherwise.

*

If it's alright with others, I can transfer it to the tldr-pages repo now. Seems like the right time now that I'm more openly prompting discussion.

Edit: https://github.com/tldr-pages/tldr-pages-test-harness

Anyone is welcome to yell at me if there are problems with it, or if in practice it's not something we're keen to maintain in the org. I don't mean to add unneccesary maintenence burden on anyone else.

15:53:25
@sbrl-57247531659847a7aff542b6:gitter.imsbrl (Starbeamrainbowlabs)
In reply to @sethi:one.ems.host

If it's alright with others, I can transfer it to the tldr-pages repo now. Seems like the right time now that I'm more openly prompting discussion.

Edit: https://github.com/tldr-pages/tldr-pages-test-harness

Anyone is welcome to yell at me if there are problems with it, or if in practice it's not something we're keen to maintain in the org. I don't mean to add unneccesary maintenence burden on anyone else.

seems fine for now! is it a work in progress, or a 'completed' thing?
18:36:04
@kbdk:matrix.orgK.B.Dharun KrishnaAdding all our clients is a WIP whereas the bats implementation is itself is complete afaik.19:02:28
@kbdk:matrix.orgK.B.Dharun Krishna* Adding all our clients is a WIP whereas the bats implementation is complete afaik.19:02:54
@sethi:one.ems.hostSeth
In reply to @sbrl-57247531659847a7aff542b6:gitter.im
seems fine for now! is it a work in progress, or a 'completed' thing?

It's still a WIP overall, but I don't expect the structure change much unless I receive substantial feedback. The things I'm actively addressing are:

  1. Working with a designer for a better compliance badges.
  2. As Dharun said, include Dockerfiles for other popular clients.
  3. Then just acting on feedback if anyone wants to comment. (Issues/PRs/complaints welcome!)

Transfering it to the org wasn't an indicator that it's done by any means, but just that to me it seems succesful as a proof-of-concept (both self-review and based on feedback in the chat) and so I wanted to move it to a shared space to encourage feedback/contributions.

21:24:13
@sbrl-57247531659847a7aff542b6:gitter.imsbrl (Starbeamrainbowlabs)
In reply to @sethi:one.ems.host

It's still a WIP overall, but I don't expect the structure change much unless I receive substantial feedback. The things I'm actively addressing are:

  1. Working with a designer for a better compliance badges.
  2. As Dharun said, include Dockerfiles for other popular clients.
  3. Then just acting on feedback if anyone wants to comment. (Issues/PRs/complaints welcome!)

Transfering it to the org wasn't an indicator that it's done by any means, but just that to me it seems succesful as a proof-of-concept (both self-review and based on feedback in the chat) and so I wanted to move it to a shared space to encourage feedback/contributions.

sounds good!
is there any way to tell why the failing clients failed at all?
23:24:00
@sethi:one.ems.hostSeth
In reply to @sbrl-57247531659847a7aff542b6:gitter.im
sounds good!
is there any way to tell why the failing clients failed at all?

Currently, the only way would be to check the last CI run and review the logs.

They're all run in a single action, so you'd have to just scroll/search for the client you're interested in:
https://github.com/tldr-pages/tldr-pages-test-harness/actions/runs/8068177164/job/22040305500

(Note, while tests are defined for optional/recommends, CI currently on runs tests for required specifications.)

I could split that up so it's easier to read logs by client. I did it this way to avoid maintaining the list of clients in another place, but it's not very practical.

23:27:28
@sethi:one.ems.hostSeth
In reply to @sbrl-57247531659847a7aff542b6:gitter.im
sounds good!
is there any way to tell why the failing clients failed at all?
*

Currently, the only way would be to check the last CI run and review the logs.

They're all run in a single action, so you'd have to just scroll/search for the client you're interested in:
https://github.com/tldr-pages/tldr-pages-test-harness/actions/runs/8068177164/job/22040305500

(Note, while tests are defined for optional/recommends, CI currently only runs tests for required specifications.)

I could split that up so it's easier to read logs by client. I did it this way to avoid maintaining the list of clients in another place, but it's not very practical.

23:28:03
@sethi:one.ems.hostSeth
In reply to @sbrl-57247531659847a7aff542b6:gitter.im
sounds good!
is there any way to tell why the failing clients failed at all?
*

Currently, the only way would be to check the last CI run and review the logs, or to run it locally.

They're all run in a single action, so you'd have to just scroll/search for the client you're interested in:
https://github.com/tldr-pages/tldr-pages-test-harness/actions/runs/8068177164/job/22040305500

(Note: While tests are defined for optional/recommends, CI currently only runs tests for required specifications. Happy to change that… I made it lenient during the proof-of-concept and figured I'd just leave it until we badges to separate different compliance levels. ^-^')

I could split that up so it's easier to read logs by client. I did it this way to avoid maintaining the list of clients in another place, but it's not very practical.

23:30:24
@sethi:one.ems.hostSeth
In reply to @sbrl-57247531659847a7aff542b6:gitter.im
sounds good!
is there any way to tell why the failing clients failed at all?
*

Currently, the only way would be to check the last CI run and review the logs, or to run it locally.

They're all run in a single action, so you'd have to just scroll/search for the client you're interested in:
https://github.com/tldr-pages/tldr-pages-test-harness/actions/runs/8068177164/job/22040305500

(Note: While tests are defined for optional/recommends, CI currently only runs tests for required specifications. Happy to change that, but I made it lenient during the proof-of-concept and figured I'd just leave it until we have badges to separate different compliance levels.)

I could split that up so it's easier to read logs by client. I did it this way to avoid maintaining the list of clients in another place, but it's not very practical.

23:30:47
@sethi:one.ems.hostSeth
In reply to @sbrl-57247531659847a7aff542b6:gitter.im
sounds good!
is there any way to tell why the failing clients failed at all?
*

Currently, the only way would be to check the last CI run and review the logs, or to run it locally.

They're all run in a single action, so you'd have to just scroll/search for the client you're interested in:
https://github.com/tldr-pages/tldr-pages-test-harness/actions/runs/8068177164/job/22040305500

(Note: While tests are defined for optional/recommends, CI currently only runs tests for required specifications. Happy to change that, but I made it lenient during the proof-of-concept and figured I'd just leave it until we have badges to separate different compliance levels.)

I could split the action up too, so it's easier to read logs by client. I did it this way to avoid maintaining the list of clients in another place, but it's not very practical to read if people want to review it via the GitHub Action.

23:31:21
@sethi:one.ems.hostSeth
In reply to @sbrl-57247531659847a7aff542b6:gitter.im
sounds good!
is there any way to tell why the failing clients failed at all?
*

Currently, the only way would be to check the last CI run and review the logs, or to run it locally.

They're all run in a single action, so you'd have to just scroll/search for the client you're interested in:
https://github.com/tldr-pages/tldr-pages-test-harness/actions/runs/8068177164/job/22040305500

(Note: While tests are defined for optional/recommends, CI currently only runs tests for required specifications. Happy to change that, but I made it lenient during the proof-of-concept and figured I'd just leave it until we have badges to separate different compliance levels.)

I could split the action up too so it's easier to read logs by client. I did it this way to avoid maintaining the list of clients in another place, but it's not very practical to read if people want to review it via the GitHub Action.

23:32:12
@sethi:one.ems.hostSeth
In reply to @sethi:one.ems.host

Currently, the only way would be to check the last CI run and review the logs.

They're all run in a single action, so you'd have to just scroll/search for the client you're interested in:
https://github.com/tldr-pages/tldr-pages-test-harness/actions/runs/8068177164/job/22040305500

(Note, while tests are defined for optional/recommends, CI currently on runs tests for required specifications.)

I could split that up so it's easier to read logs by client. I did it this way to avoid maintaining the list of clients in another place, but it's not very practical.

If you're alright with checking it locally, I can update the README to clarify both ways to run it locally. That might be easier that digging through the CI logs for now.

If you have the client installed, you can do so with:

PATH_TO_TLDR_CLIENT={{path/to/binary}} make validate

If you don't have it installed, you can do:

# specify which client based on the directory names in dockerfiles/
CLIENT_NAME=tldr-c-client
docker build -t tldr-test:$CLIENT_NAME -f dockerfiles/$CLIENT_NAME/Dockerfile .
docker run -it tldr-test:$CLIENT_NAME make validate

You can replace validate with validate-level-2 or validate-level-3 to turn on optional tests.

23:40:47
@sethi:one.ems.hostSeth
In reply to @sethi:one.ems.host

Currently, the only way would be to check the last CI run and review the logs.

They're all run in a single action, so you'd have to just scroll/search for the client you're interested in:
https://github.com/tldr-pages/tldr-pages-test-harness/actions/runs/8068177164/job/22040305500

(Note, while tests are defined for optional/recommends, CI currently on runs tests for required specifications.)

I could split that up so it's easier to read logs by client. I did it this way to avoid maintaining the list of clients in another place, but it's not very practical.

*

If you're alright with checking it locally, I can update the README to clarify both ways to run it locally. That might be easier that digging through the CI logs for now.

If you have the client installed, you can do so with:

PATH_TO_TLDR_CLIENT={{path/to/binary}} make validate

If you don't have it installed, you can do:

# specify which client based on the directory names in dockerfiles/
CLIENT_NAME=tldr-c-client
docker build -t tldr-test:$CLIENT_NAME -f dockerfiles/$CLIENT_NAME/Dockerfile .
docker run -it tldr-test:$CLIENT_NAME make validate

(I should convert the Docker version to a make command. ^-^')

You can replace validate with validate-level-2 or validate-level-3 to turn on optional tests.

23:41:46
@sethi:one.ems.hostSeth
In reply to @sethi:one.ems.host

Currently, the only way would be to check the last CI run and review the logs.

They're all run in a single action, so you'd have to just scroll/search for the client you're interested in:
https://github.com/tldr-pages/tldr-pages-test-harness/actions/runs/8068177164/job/22040305500

(Note, while tests are defined for optional/recommends, CI currently on runs tests for required specifications.)

I could split that up so it's easier to read logs by client. I did it this way to avoid maintaining the list of clients in another place, but it's not very practical.

*

If you're alright with checking it locally, I can update the README to clarify both ways to run it locally. That might be easier that digging through the CI logs for now.

If you have the client installed, you can do so with:

PATH_TO_TLDR_CLIENT={{path/to/binary}} make validate

If you don't have it installed, you can do:

# specify which client based on the directory names in dockerfiles/
CLIENT_NAME=tldr-c-client
docker build -t tldr-test:$CLIENT_NAME -f dockerfiles/$CLIENT_NAME/Dockerfile .
docker run -it tldr-test:$CLIENT_NAME make validate

(I should convert the Docker way to a make command. ^-^')

You can replace validate with validate-level-2 or validate-level-3 to turn on optional tests.

23:41:53
@sethi:one.ems.hostSeth
In reply to @sbrl-57247531659847a7aff542b6:gitter.im
sounds good!
is there any way to tell why the failing clients failed at all?
*

Currently, the only way would be to check the last CI run and review the logs, or to run it locally.

They're all run in a single action, so you'd have to scroll/search for the client you're interested in:
https://github.com/tldr-pages/tldr-pages-test-harness/actions/runs/8068177164/job/22040305500

(Note: While tests are defined for optional/recommends, CI currently only runs tests for required specifications. Happy to change that, but I made it lenient during the proof-of-concept and figured I'd just leave it until we have badges to separate different compliance levels.)

I could split the action up too so it's easier to read logs by client. I did it this way to avoid maintaining the list of clients in another place, but it's not very practical to read if people want to review it via the GitHub Action.

23:42:24
28 Feb 2024
@sethi:one.ems.hostSeth
In reply to @sethi:one.ems.host

Currently, the only way would be to check the last CI run and review the logs.

They're all run in a single action, so you'd have to just scroll/search for the client you're interested in:
https://github.com/tldr-pages/tldr-pages-test-harness/actions/runs/8068177164/job/22040305500

(Note, while tests are defined for optional/recommends, CI currently on runs tests for required specifications.)

I could split that up so it's easier to read logs by client. I did it this way to avoid maintaining the list of clients in another place, but it's not very practical.

*

If you're alright with checking it locally, I can update the README to clarify both ways to run it locally. That might be easier that digging through the CI logs for now.

If you have the client installed, you can do so with:

PATH_TO_TLDR_CLIENT={{path/to/binary}} make validate

If you don't have it installed but have Docker setup, you can do:

CLIENT={client-name} make docker-validate

You can append validate/docker-validate with -level-2 or -level-3 to turn on optional tests.

00:24:37
@sethi:one.ems.hostSeth
In reply to @sethi:one.ems.host

Currently, the only way would be to check the last CI run and review the logs.

They're all run in a single action, so you'd have to just scroll/search for the client you're interested in:
https://github.com/tldr-pages/tldr-pages-test-harness/actions/runs/8068177164/job/22040305500

(Note, while tests are defined for optional/recommends, CI currently on runs tests for required specifications.)

I could split that up so it's easier to read logs by client. I did it this way to avoid maintaining the list of clients in another place, but it's not very practical.

*

If you're alright with checking it locally, I can update the README to clarify both ways to run it locally. That might be easier that digging through the CI logs for now.

If you have the client installed, you can do so with:

PATH_TO_TLDR_CLIENT={{path/to/binary}} make validate

If you don't have it installed but have Docker setup, you can check the list of clients supported in the dockerfiles/ directory, then do:

CLIENT={client-name} make docker-validate

You can append validate/docker-validate with -level-2 or -level-3 to turn on optional tests.

00:26:16
@sethi:one.ems.hostSeth
In reply to @sethi:one.ems.host

Currently, the only way would be to check the last CI run and review the logs.

They're all run in a single action, so you'd have to just scroll/search for the client you're interested in:
https://github.com/tldr-pages/tldr-pages-test-harness/actions/runs/8068177164/job/22040305500

(Note, while tests are defined for optional/recommends, CI currently on runs tests for required specifications.)

I could split that up so it's easier to read logs by client. I did it this way to avoid maintaining the list of clients in another place, but it's not very practical.

*

If you're alright with checking it locally, that might be easier that digging through the CI logs for now.

If you have the client installed locally, you can do so with:

PATH_TO_TLDR_CLIENT={{path/to/binary}} make validate

If you don't have it installed but have Docker setup, you can check the list of clients supported in the dockerfiles/ directory, then do:

CLIENT={client-name} make docker-validate

Demo: https://github.com/tldr-pages/tldr-pages-test-harness/pull/3

You can append validate/docker-validate with -level-2 or -level-3 to turn on optional tests.

00:27:25
@sethi:one.ems.hostSeth
In reply to @sethi:one.ems.host

Currently, the only way would be to check the last CI run and review the logs.

They're all run in a single action, so you'd have to just scroll/search for the client you're interested in:
https://github.com/tldr-pages/tldr-pages-test-harness/actions/runs/8068177164/job/22040305500

(Note, while tests are defined for optional/recommends, CI currently on runs tests for required specifications.)

I could split that up so it's easier to read logs by client. I did it this way to avoid maintaining the list of clients in another place, but it's not very practical.

*

If you're alright with checking it locally, that might be easier than digging through the CI logs for now.

If you have the client installed locally, you can do so with:

PATH_TO_TLDR_CLIENT={{path/to/binary}} make validate

If you don't have it installed but have Docker setup, you can check the list of clients supported in the dockerfiles/ directory, then do:

CLIENT={client-name} make docker-validate

Demo: https://github.com/tldr-pages/tldr-pages-test-harness/pull/3

You can append validate/docker-validate with -level-2 or -level-3 to turn on optional tests.

00:32:40
@sbrl-57247531659847a7aff542b6:gitter.imsbrl (Starbeamrainbowlabs)Ah, I see. I only ask because having a live badge as to whether each client passes or fails is fine, but client authors will want a documented way to check how they client fails00:56:11
@sbrl-57247531659847a7aff542b6:gitter.imsbrl (Starbeamrainbowlabs) * Ah, I see. I only ask because having a live badge as to whether each client passes or fails is fine, but client authors will likely want a documented way to check how they client fails 00:56:21

Show newer messages


Back to Room ListRoom Version: 6