solve this https://w.trhou.se/7000qz679f . i want to remove the last item on the list . tell me what is wrong with my co

// You should avoid getting the element like this because if you add another ul element before the
// one that you are trying to get, it will end up getting the different ul element and will break your code
// I will post the correct way to get the element below.
let ul=document.getElementsByTagName('ul')[2];
// this would work if you had selected the correct ul element. The above ul element was from your
// nav bar, thus selecting the last li element which happens to be 'Contact'. 
let li = document.querySelector('li:last-child');

// app.js
let input = document.querySelector('.inputSkill');
let button = document.querySelector('.addSkill');
let p = document.querySelector('.musab');

// I moved the ul and button2 variable up here
// Instead of defining the ul element 2 times, you can just define it up here once.
let ul = document.querySelector('ul.skills');
let button2 = document.querySelector('.removeItem');

button.addEventListener('click', ()=>{
// I added this if statement so that it does not add empty or null li elements
  if (input.value !== '' || input.value === null) {
      let li = document.createElement('li');
      li.textContent = input.value;
      ul.appendChild(li);
      input.value= '';
  }
});

// I added an e (event.target), so that we check to if the event.target is an actual button
button2.addEventListener('click', (e)=>{
  // Just in case we are checking to see if event.target is button
  // with the tagName of BUTTON, then if it's true, we remove the lastElementChild from ul
  if (e.target.tagName === 'BUTTON') ul.removeChild(ul.lastElementChild)
   // ul.lastElementChild is the last li element that you want to remove
});